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.

Sign up for GitHub

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

Jump to bottom

feat: remove blobs beta tags #6392

Merged
merged 4 commits into from
Feb 20, 2024
Merged

feat: remove blobs beta tags #6392

merged 4 commits into from
Feb 20, 2024

Conversation

kathmbeck
Copy link
Contributor

🎉 Thanks for submitting a pull request! 🎉

Summary

Fixes https://linear.app/netlify/issue/ADN-335/remove-beta-flag-from-blobs-cli-reference
For GA release

For us to review and ship your PR efficiently, please perform the following steps:

  • Open a bug/issue before writing your code 🧑‍💻. This ensures we can discuss the changes and get feedback from everyone that should be involved. If you`re fixing a typo or something that`s on fire 🔥 (e.g. incident related), you can skip this step.
  • Read the contribution guidelines 📖. This ensures your code follows our style guide and
    passes our tests.
  • Update or add tests (if any source code was changed or added) 🧪
  • Update or add documentation (if features were changed or added) 📝
  • Make sure the status checks below are successful ✅

A picture of a cute animal (not mandatory, but encouraged)

@kathmbeck kathmbeck requested review from a team as code owners February 20, 2024 14:57
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 20, 2024 •
edited

📊 Benchmark results

Comparing with b0d8d67

  • Dependency count: 1,283 (no change)
  • Package size: 282 MB (no change)
  • Number of ts-expect-error directives: 1,180 (no change)

khendrikse
khendrikse previously approved these changes Feb 20, 2024
View reviewed changes
Copy link
Contributor

@khendrikse khendrikse left a comment

Choose a reason for hiding this comment

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

Awesome!

klavavej
klavavej approved these changes Feb 20, 2024
View reviewed changes
Copy link

@klavavej klavavej left a comment

Choose a reason for hiding this comment

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

LGTM! 💖

@kathmbeck kathmbeck merged commit aa759a4 into main Feb 20, 2024
36 checks passed
@kathmbeck kathmbeck deleted the remove-beta-tags branch February 20, 2024 18:04
@token-generator-app token-generator-app bot mentioned this pull request Feb 20, 2024
Merged
kathmbeck added a commit that referenced this pull request Feb 20, 2024
@kathmbeck
Revert "feat: remove blobs beta tags (#6392)"
f666b35 
This reverts commit aa759a4.
@kathmbeck kathmbeck mentioned this pull request Feb 20, 2024
Merged
kathmbeck added a commit that referenced this pull request Feb 20, 2024
@kathmbeck
Revert "feat: remove blobs beta tags (#6392)" (#6397)
0c91f20 
This reverts commit aa759a4.
kathmbeck added a commit that referenced this pull request Feb 20, 2024
@kathmbeck
Revert "Revert "feat: remove blobs beta tags (#6392)" (#6397)"
3224aa9 
This reverts commit 0c91f20.
kathmbeck added a commit that referenced this pull request Mar 14, 2024
@kathmbeck
Revert "Revert "feat: remove blobs beta tags (#6392)" (#6397)" (#6398)
10965ab 
This reverts commit 0c91f20.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@khendrikse khendrikse khendrikse approved these changes

@klavavej klavavej klavavej approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@kathmbeck @khendrikse @klavavej