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 Github API from generating hab release notes #3769

Merged
merged 3 commits into from
Mar 15, 2022
Merged

Remove Github API from generating hab release notes #3769

merged 3 commits into from
Mar 15, 2022

Conversation

IanMadd
Copy link
Contributor

@IanMadd IanMadd commented Mar 15, 2022

Signed-off-by: Ian Maddaus [email protected]

Description

We exceeded the GitHub API rate limit by grabbing the release versions on every build. So instead of relying on that, I just copied the version numbers into a file and we'll use that instead.

Definition of Done

Issues Resolved

[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussion that's relevant]

Related PRs

Check List

  • Spell Check
  • Local build
  • Examine the local build
  • All tests pass

@IanMadd IanMadd requested a review from a team as a code owner March 15, 2022 19:24
@netlify
Copy link

netlify bot commented Mar 15, 2022

✔️ Deploy Preview for chef-web-docs ready!

🔨 Explore the source changes: 76d0e53

🔍 Inspect the deploy log: https://app.netlify.com/sites/chef-web-docs/deploys/6230fd9704a14c00089012c9

😎 Browse the preview: https://deploy-preview-3769--chef-web-docs.netlify.app/release_notes_habitat

Signed-off-by: Ian Maddaus <[email protected]>
@IanMadd IanMadd merged commit 735cbf0 into main Mar 15, 2022
@IanMadd IanMadd deleted the im/hab_rn branch March 15, 2022 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant