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

Contributing.md update #3799

Closed
wants to merge 37 commits into from
Closed

Contributing.md update #3799

wants to merge 37 commits into from

Conversation

enriquediaz
Copy link
Contributor

Harvard has drafted this update to the document for contributors to Mirador, which includes additional information of potential use to them, along with some link updates to keep up with the times.

@lutzhelm
Copy link
Contributor

lutzhelm commented Nov 2, 2023

Hi Enrique, it looks like your PR branch is based on the mui5 branch, as it contains most commits from that branch. Could you update the PR to a branch based on master?

git rebase --onto master mui5 contributing.md-update
git push --force origin contributing.md-update

@enriquediaz
Copy link
Contributor Author

Thank you for catching that @lutzhelm! I'm experiencing difficulty with this branch so I will close this PR, clean things up and retry separately.

@enriquediaz enriquediaz closed this Nov 2, 2023
@enriquediaz enriquediaz deleted the contributing.md-update branch November 2, 2023 18:15
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.

3 participants