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

fix: Bump manifest only on main #14

Merged
merged 2 commits into from
May 25, 2023
Merged

fix: Bump manifest only on main #14

merged 2 commits into from
May 25, 2023

Conversation

Anush008
Copy link
Member

@Anush008 Anush008 commented May 25, 2023

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation Update
  • 🎨 Style
  • πŸ§‘β€πŸ’» Code Refactor
  • πŸ”₯ Performance Improvements
  • βœ… Test
  • πŸ€– Build
  • πŸ” CI
  • πŸ“¦ Chore (Release)
  • ⏩ Revert

Description

This PR fixes the issue mentioned in open-sauced/ai#121, by bumping the manifest.json version number only on main with integer-only version numbers.

Demo manifest bump on main: Anush008@3098723

No manifest bump on beta: Anush008@eb6a98c

Related Tickets & Documents

Merging this will contribute to resolving open-sauced/ai#121.

Mobile & Desktop Screenshots/Recordings

Added tests?

  • πŸ‘ yes
  • πŸ™… no, because they aren't needed
  • πŸ™‹ no, because I need help

Added to documentation?

  • πŸ“œ README.md
  • πŸ““ docs.opensauced.pizza
  • πŸ• dev.to/opensauced
  • πŸ“• storybook
  • πŸ™… no documentation needed

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

Copy link
Contributor

@0-vortex 0-vortex left a comment

Choose a reason for hiding this comment

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

LGTM! πŸ‘

@0-vortex 0-vortex merged commit 11bec17 into open-sauced:main May 25, 2023
github-actions bot pushed a commit that referenced this pull request May 25, 2023
## [2.1.1](v2.1.0...v2.1.1) (2023-05-25)

### πŸ› Bug Fixes

* bump manifest only on main ([#14](#14)) ([11bec17](11bec17))
@github-actions
Copy link

πŸŽ‰ This PR is included in version 2.1.1 πŸŽ‰

The release is available on:

Your semantic-release bot πŸ“¦πŸš€

@Anush008 Anush008 deleted the patch-1 branch May 25, 2023 07:55
github-actions bot pushed a commit to OgDev-01/release that referenced this pull request May 15, 2024
## 1.0.0 (2024-05-15)

### ⚠ BREAKING CHANGES

* adjust for

### πŸ” Continuous Integration

* add better release testing workflows ([77c6515](77c6515))

### πŸ› Bug Fixes

* bump manifest only on main ([open-sauced#14](https://github.com/ogdev-01/release/issues/14)) ([11bec17](11bec17))
* correct action pre-release version ([143c4c7](143c4c7))
* correct breaking changes detection ([open-sauced#6](https://github.com/ogdev-01/release/issues/6)) ([686d8fa](686d8fa)), closes [open-sauced#1](https://github.com/ogdev-01/release/issues/1)
* correct glob for public assets and supabase ([cff5c94](cff5c94))
* correct license plugin and refactor all tooling ([9779237](9779237))
* correct release remotes ([11a1763](11a1763))
* glob pattern ([2f85058](2f85058))
* remove limited license plugin ([open-sauced#8](https://github.com/ogdev-01/release/issues/8)) ([fc06582](fc06582)), closes [open-sauced#7](https://github.com/ogdev-01/release/issues/7)
* upgrade node 18 in release workflows ([92a2f86](92a2f86))

### πŸ“ Documentation

* adjust plugins in readme ([13777e1](13777e1))
* NPM_PACKAGE_ROOT README.md ([1d80114](1d80114))
* skip changelog README.md ([fd28263](fd28263))

### πŸ• Features

* add hot open sauced supabase feature integration ([3d8cfbf](3d8cfbf))
* Adds SKIP_NPM_PUBLISH and SKIP_DOCKER_PUBLISH ([c89740d](c89740d))
* bump pkg, pkg-lock subdir ([c2109ee](c2109ee))
* commit npm-shrinkwrap.json ([5ce8e0e](5ce8e0e))
* configure variables ([af2eac1](af2eac1))
* initial commit ([ed3c774](ed3c774))
* manifest version value bump ([open-sauced#13](https://github.com/ogdev-01/release/issues/13)) ([9f4cd6f](9f4cd6f))
* npm package root path ([7048dd6](7048dd6))
* remove google analyser plugin from config ([3e5f571](3e5f571))
* setup release for ogdev ([dabdc21](dabdc21))
* skip changelog gen ([c7b6e33](c7b6e33))
* work around CVE-2022-24765 changes and update to esm release binary ([57c4acc](57c4acc)), closes [/github.blog/2022-04-12-git-security-vulnerability-announced/#cve-2022-24765](https://github.com/ogdev-01//github.blog/2022-04-12-git-security-vulnerability-announced//issues/cve-2022-24765) [open-sauced#10](https://github.com/ogdev-01/release/issues/10)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants