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

Remove CHANGELOG.md #1011

Merged
merged 3 commits into from
Apr 14, 2022
Merged

Remove CHANGELOG.md #1011

merged 3 commits into from
Apr 14, 2022

Conversation

jonaharagon
Copy link
Member

This is just additional work, I didn't realize GitHub would automatically generate such changelogs with our new release strategy :)

We just have to make sure going forward that PRs are titled appropriately/clearly since they are used when auto-generating release notes.

See releases for an example of the type of generated release notes we'd just use instead.

@netlify
Copy link

netlify bot commented Apr 13, 2022

Deploy Preview for privacyguides ready!

Name Link
🔨 Latest commit 1a9da46
🔍 Latest deploy log https://app.netlify.com/sites/privacyguides/deploys/62576cba03c196000877d51c
😎 Deploy Preview https://deploy-preview-1011--privacyguides.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@realguyman
Copy link
Contributor

I forgot about this feature. I couldn't seem to get it to work on my personal projects, though. Seems to be working just fine. :)

@jonaharagon
Copy link
Member Author

@realguyman I believe it only generates based on PRs, so it works here since we have a PR heavy workflow.

@jonaharagon jonaharagon enabled auto-merge (squash) April 14, 2022 00:37
@jonaharagon jonaharagon merged commit a379164 into main Apr 14, 2022
@jonaharagon jonaharagon deleted the pr-changelog branch April 14, 2022 00:38
@dngray dngray added the l10n Localization label Apr 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n Localization
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants