Skip to content
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

Update license #299

Merged
merged 7 commits into from
Nov 3, 2021
Merged

Update license #299

merged 7 commits into from
Nov 3, 2021

Conversation

KatherineCox
Copy link
Contributor

Updating license from MIT to CC-BY (closes #296)

Copy link
Collaborator

@cansavvy cansavvy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems reasonable to me!

@KatherineCox
Copy link
Contributor Author

@cansavvy looks like it's having trouble building?

for build-all

Run actions/checkout@v2
Error: Input required and not supplied: token

Possibly because I forked it? I don't know much about GitHub actions and tokens.

@cansavvy
Copy link
Collaborator

cansavvy commented Nov 1, 2021

It's complaining about the GitHub secret, but not sure why. It shouldn't have expired, but I'm not sure why its not working. I'll try refreshing the token.

@cansavvy
Copy link
Collaborator

cansavvy commented Nov 1, 2021

Render isn't failing on #295 ? I'm going to push a fresh commit and see if that fixes it.

@cansavvy
Copy link
Collaborator

cansavvy commented Nov 1, 2021

OH, I see its because this is a forked branch so it does not have those github secrets.

@cansavvy
Copy link
Collaborator

cansavvy commented Nov 1, 2021

@KatherineCox We'll need to transfer over your changes to this repository's branches, or if you supply a GitHub PAT to your own forked repository that should fix it too, since you have jhudsl permissions.

@cansavvy cansavvy mentioned this pull request Nov 1, 2021
@KatherineCox
Copy link
Contributor Author

@cansavvy do I have permission to make a branch? (is there a way I can check my permissions for a repository?) If so, I can redo the PR from a branch and close this one. It's easy enough to recreate, just the two changes.

@cansavvy
Copy link
Collaborator

cansavvy commented Nov 2, 2021

@KatherineCox You should have permissions to make a PR from a new branch if you are a part of the jhudsl organization.

@KatherineCox
Copy link
Contributor Author

Hmm, if I try to edit the README (through the GitHub GUI) it says

You’re making changes in a project you don’t have write access to. Submitting a change will write it to a new branch in your fork KatherineCox/DaSL_Course_Template_Bookdown, so you can send a pull request.

@cansavvy
Copy link
Collaborator

cansavvy commented Nov 3, 2021

You’re making changes in a project you don’t have write access to. Submitting a change will write it to a new branch in your fork KatherineCox/DaSL_Course_Template_Bookdown, so you can send a pull request.

Hmm... This is perplexing. You are a part of jhudsl. It may have to do with permissions at the organization level, because I can't see anything specific to this repository. I'll look into it.

@cansavvy
Copy link
Collaborator

cansavvy commented Nov 3, 2021

@KatherineCox In your example here, did you try to edit by making a new branch? Direct merges to main are prohibited.

You’re making changes in a project you don’t have write access to. Submitting a change will write it to a new branch in your fork KatherineCox/DaSL_Course_Template_Bookdown, so you can send a pull request.

@cansavvy
Copy link
Collaborator

cansavvy commented Nov 3, 2021

@KatherineCox In your example here, did you try to edit by making a new branch? Direct merges to main are prohibited.

You’re making changes in a project you don’t have write access to. Submitting a change will write it to a new branch in your fork KatherineCox/DaSL_Course_Template_Bookdown, so you can send a pull request.

No, this isn't it, looks like you have different permissions to jhudsl. But if you don't mind, I would like to use this as an opportunity to see what the best way is to incorporate changes from someone who doesn't have the same permissions.

@cansavvy
Copy link
Collaborator

cansavvy commented Nov 3, 2021

Thanks for your patience and help, @KatherineCox , this ended up being a good thing to figure out so more people can contribute in the future! Turns out the token is not needed in that particular instance, so I got rid of it so no matter what permissions levels people have they can file a PR!

@KatherineCox
Copy link
Contributor Author

Fine by me :) Just let me know what you need me to do. And I may need some hand-holding if I need to do anything with tokens, I don't really know how they work.

@cansavvy
Copy link
Collaborator

cansavvy commented Nov 3, 2021

Well, the preview builds properly, it just doesn't have permissions to write the comment with the links. There's a way around this that involves writing a github actions that triggers with pull_request_target: synchronize but I think this is a job for another day. The payoff is not worth how much its going to further obfuscate the github actions setup

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update License
2 participants