Skip to content

Build: upgrade commonmark to 0.9.1 #9563

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

Merged
merged 1 commit into from
Sep 1, 2022
Merged

Conversation

humitos
Copy link
Member

@humitos humitos commented Aug 30, 2022

Currently, we are installing an older version of commonmark that has some
problems with newer versions of Python. Upgrading it to 0.9.1 seems to solve
this problem.

See #9562 for a detailed explanation of the problem


📚 Documentation previews 📚

Currently, we are installing an older version of `commonmark` that has some
problems with newer versions of Python. Upgrading it to 0.9.1 seems to solve
this problem.

See #9562 for a detailed explanation of the problem
@humitos humitos requested a review from a team as a code owner August 30, 2022 07:43
@humitos humitos requested a review from stsewd August 30, 2022 07:43
@humitos humitos added the PR: hotfix Pull request applied as hotfix to release label Aug 30, 2022
Copy link
Member

@stsewd stsewd left a comment

Choose a reason for hiding this comment

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

@humitos
Copy link
Member Author

humitos commented Aug 31, 2022

What's the workflow for adding commits on a PR that it was already hotfixed into rel/relcorp now that we are using squash & merge? I'd assume that we will get conflicts when doing a new release next week? cc @stsewd @agjohnson

@agjohnson
Copy link
Contributor

We could also hotfix the new commits on this PR, or alternatively (and probably my suggestion) move on to a new PR -- that is, treat this PR as hotfix, merge it, and follow up with a PR that is not hotfix for the new commits.

@humitos humitos merged commit 0afdbe4 into main Sep 1, 2022
@humitos humitos deleted the humitos/upgrade-commonmark branch September 1, 2022 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: hotfix Pull request applied as hotfix to release
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants