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

Configure giles to check PRs #70

Merged
merged 2 commits into from Jun 21, 2022
Merged

Conversation

samaloney
Copy link
Collaborator

PR Description

Configure bot to check PR have proper change logs

@DanRyanIrish DanRyanIrish changed the title Configure giles to check RPs Configure giles to check PRs Jun 15, 2022
pyproject.toml Outdated
Comment on lines 30 to 34
help_url = "https://github.com/sunpy/sunxspex/blob/main/changelog/README.rst"

changelog_missing_long = "There isn't a changelog file in this pull request. Please add a changelog file to the `changelog/` directory following the instructions in the changelog [README](https://github.com/sunpy/sunpy/blob/main/changelog/README.rst)."

type_incorrect_long = "The changelog file you added is not one of the allowed types. Please use one of the types described in the changelog [README](https://github.com/sunpy/sunpy/blob/main/changelog/README.rst)"
Copy link
Member

Choose a reason for hiding this comment

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

The sunxspex default branch is still master (that can probably be changed?) so the help_url is currently 404. The updated towncrier types here seem to correspond to the sunpy/sunpy version which is out of sync with the sunpy/sunxspex changelog readme (changelog/README.rst). Maybe this readme should also be re-synced with the sunpy version, and the URLs in changelog_missing_long and type_incorrect_long above updated to point to the README within the sunxspex repo.

@samaloney samaloney merged commit 818ae4f into sunpy:master Jun 21, 2022
@samaloney samaloney deleted the feat-enable-giles branch June 21, 2022 08:56
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.

None yet

4 participants