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

Add header and footer #1050

Merged
merged 2 commits into from Feb 1, 2022
Merged

Add header and footer #1050

merged 2 commits into from Feb 1, 2022

Conversation

mkurz
Copy link
Contributor

@mkurz mkurz commented Feb 1, 2022

Fixes #1047

@mkurz
Copy link
Contributor Author

mkurz commented Feb 1, 2022

@jetersen After adding tests this is ready to review and hopefully merge. Thanks!

@jetersen jetersen merged commit a8445e8 into release-drafter:master Feb 1, 2022
@jetersen
Copy link
Member

jetersen commented Feb 1, 2022

Again thank you for your contribution 🙌

@mkurz
Copy link
Contributor Author

mkurz commented Feb 1, 2022

Again thank you!

@mkurz mkurz deleted the header_footer branch February 1, 2022 21:51
@jetersen
Copy link
Member

jetersen commented Feb 1, 2022

I'll put a tag on first thing tomorrow 😉 but bedtime is calling 😊

@mkurz
Copy link
Contributor Author

mkurz commented Feb 1, 2022

👍 Actually there is a bug regarding _extends in https://github.com/probot/octokit-plugin-config, I am working on a fix right now, so maybe we can wait with the release. Let's talk tomorrow.

@mkurz
Copy link
Contributor Author

mkurz commented Feb 1, 2022

@jetersen That is the bug I was talking about which also affects release-drafter: probot/octokit-plugin-config#166
Fix is here: probot/octokit-plugin-config#167

@jetersen
Copy link
Member

jetersen commented Feb 3, 2022

whether dependency supports .github repo _extends or not have nothing to do with header and footer? 🤔

@mkurz
Copy link
Contributor Author

mkurz commented Feb 3, 2022

Actually we want to put footer in a main config and a couple of configs will extend from it, all placed within the .github repo. So for us we need both to work correctly, so that's why I thought lets wait for the octokit-plugin-config fix and put everything in one release.
However of course technically they are not related.
AFAIK releasing is just pushing tag, feel free to do so. However as soon as the octokit-plugin-config fix lands it would be nice to cut another release. Thanks!

@jetersen
Copy link
Member

jetersen commented Feb 3, 2022

a release is only a tag away so meh 😅
See: https://github.com/release-drafter/release-drafter/blob/master/.github/workflows/release.yml

@jetersen jetersen added the type: feature New feature or request label Feb 3, 2022
@mkurz
Copy link
Contributor Author

mkurz commented Feb 4, 2022

@jetersen so we have a new release of octokit-plugin-config: https://github.com/probot/octokit-plugin-config/releases/tag/v1.1.5
Can you also release a new relase drafter version with that dependency? And also move the v5 tag? Thanks!

@jetersen
Copy link
Member

jetersen commented Feb 6, 2022

@mkurz https://github.com/release-drafter/release-drafter/releases/tag/v5.18.1

@mkurz
Copy link
Contributor Author

mkurz commented Feb 6, 2022

@jetersen Thank you very much! Have a nice weekend!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add header and footer template configs
2 participants