-
Notifications
You must be signed in to change notification settings - Fork 5
Add join page #20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add join page #20
Conversation
|
Thanks for doing this. |
|
Build errors here and #21 appear to be problems with permissions. There's probably some secret thing that we can add to the build configuration, but for now I'm just going to add @emiller88 to the repo with write permission and run them again. |
|
Hmm, maybe not. @tlienart would you mind taking a look? https://github.com/BioJulia/biojulia.github.io/actions/runs/5828128326/job/15821241791?pr=20#step:4:478 |
|
the issue is something to do with tokens some notes here from 2020 on the github-pages-action repo; given that this PR: https://github.com/BioJulia/biojulia.github.io/actions/runs/5833801346 worked fine, can you try just merging stuff and seeing if it works? What might happen is that the preview path is faulty (e.g. it tries to do something with a non-existing branch, I'd need to check) but the rest works fine. |
|
Yeah, I can build it locally, and clearly it works on merging. I also works if the branch originates from this repo (eg BioJulia/biojulia.github.io#17 (comment)). But it would be awesome to have a way for external collaborators to make contributions w/o write permissions and still get build previews. [Feature request] because you obviously don't have enough on your plate 😆 |
|
No it's good to know that the preview path doesn't work properly, GitHub actions are very powerful but I'm still a beginner when it comes to those so im not surprised there are issues; definitely something to look into as your use case is one that I'd expect many people to have (eg the Julia website folks..) |
Let's see if this works |
|
Partial success. It runs, but still no preview URL 😞 |
|
🤔 well, that's a kind of progress... |
|
what may have happened is that on the last trigger the build action saw "no change" so that there was effectively "nothing new to commit". Could I suggest maybe trying to re-trigger with a dummy commit where you change a word or a (I need to do some more thorough investigation of this preview path with external contributors but I don't really have the time atm to dig through that... sorry) |
|
Have we confirmed that it works from a non-fork branch as well? |
|
Let's find out: BioJulia/biojulia.github.io#24 |
|
Cool, that's progress! Let's follow up in #23, and we can focus on the join page here now. |
|
It's a start, if there's any outstanding issues let me know and we can either address before merging this or follow up in another PR! |
|
@emiller88 Can you resolve the conflict here? I can't remember which edit on the header came first / is preferred. I'm not picky |
|
Done! |
Moved some stuff around. I'd like to make it pop a little bit more and direct people's eyes of wear to look like https://nf-co.re/join
Creating the PR looking for feedback!