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

prepare release #25

Closed
wants to merge 3 commits into from
Closed

prepare release #25

wants to merge 3 commits into from

Conversation

fghaas
Copy link
Contributor

@fghaas fghaas commented Oct 25, 2023

  • chore: Explicitly switch gitlint to regex "search" semantics
  • fix: Bump the Tutor version dependency to >=16.1.4
  • build: Version 3.0.0

gitlint previously used regex "match" semantics, where it would
evaluate regexes for ignore-by-title and ignore-body-lines from the
beginning of a line. It is switching to "search" semantics, meaning
the regex is applied against the whole line.

We are already using ignore-by-title and ignore-body-lines with
anchored regexes (i.e. ones starting with "^"), so the only thing that
we still need to set is regex-style-search in the "general" section,
in order to silence the warning.

Reference:
https://jorisroovers.github.io/gitlint/configuration/#regex-style-search
We rely on the Tutor OPENEDX_COMMON_VERSION variable to determine
the tag (or branch) to fetch for the Tubular repository.

In Tutor 16.1.4, the default value for OPENEDX_COMMON_VERSION changed
from open-release/palm.2 to open-release/palm.3. This means we now
need a version bump in our own install_requires list, so that we
require Tutor>=16.1.4.

Since this breaks compatibility with Open edX Olive (and will thus
require a major release bump on our part), update the README
accordingly.

Reference:
overhangio/tutor#923
@fghaas fghaas closed this Oct 25, 2023
@fghaas fghaas deleted the prepare-release branch October 27, 2023 09:02
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.

1 participant