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

feat: add automated release flow #866

Merged
merged 1 commit into from
Jan 19, 2022
Merged

feat: add automated release flow #866

merged 1 commit into from
Jan 19, 2022

Conversation

peterdeme
Copy link
Contributor

Add automated release flow

@github-actions
Copy link
Contributor

github-actions bot commented Jan 13, 2022

Size Change: 0 B

Total Size: 282 kB

ℹ️ View Unchanged
Filename Size Change
dist/browser.es.js 61.5 kB 0 B
dist/browser.full-bundle.min.js 34.2 kB 0 B
dist/browser.js 62.2 kB 0 B
dist/index.es.js 61.6 kB 0 B
dist/index.js 62.3 kB 0 B

compressed-size-action

@peterdeme peterdeme force-pushed the automated_release branch 2 times, most recently from 6b0dc96 to 881aa85 Compare January 13, 2022 16:51
Copy link
Contributor

@mahboubii mahboubii left a comment

Choose a reason for hiding this comment

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

good stuff 👍

.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/initiate_release.yml Outdated Show resolved Hide resolved
.github/workflows/initiate_release.yml Show resolved Hide resolved
@peterdeme peterdeme force-pushed the automated_release branch 11 times, most recently from 7b243f7 to 589c99e Compare January 14, 2022 12:00
- uses: ./.github/actions/setup-node

- uses: wagoid/commitlint-github-action@v4
Copy link
Contributor

Choose a reason for hiding this comment

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

so this linter verifies all the commits right? we dont really need to do this since we always squash the commits and merge into master, we need a linter to confirm the pull request title which is used for the commit message in upstream

Copy link
Contributor Author

Choose a reason for hiding this comment

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

how about now @mahboubii? 🙂

Copy link
Contributor

Choose a reason for hiding this comment

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

looks good, thanks!

@peterdeme peterdeme force-pushed the automated_release branch 8 times, most recently from ef46699 to f978a4e Compare January 14, 2022 15:37
@peterdeme peterdeme force-pushed the automated_release branch 5 times, most recently from 12cce45 to 0eeeb5b Compare January 14, 2022 15:57
ferhatelmas
ferhatelmas previously approved these changes Jan 18, 2022
Copy link
Contributor

@ferhatelmas ferhatelmas left a comment

Choose a reason for hiding this comment

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

LGTM

@ferhatelmas
Copy link
Contributor

ferhatelmas commented Jan 19, 2022

@peterdeme token is there in CI, we can merge. Also, could you remove Amin from codeowners?

AnatolyRugalev
AnatolyRugalev previously approved these changes Jan 19, 2022
@ferhatelmas ferhatelmas dismissed stale reviews from AnatolyRugalev and themself via ea78505 January 19, 2022 16:23
@ferhatelmas ferhatelmas force-pushed the automated_release branch 2 times, most recently from ea78505 to 1a72b67 Compare January 19, 2022 16:24
@ferhatelmas ferhatelmas removed the request for review from mahboubii January 19, 2022 16:24
@ferhatelmas ferhatelmas merged commit 9b81974 into master Jan 19, 2022
@ferhatelmas ferhatelmas deleted the automated_release branch January 19, 2022 16:29
ffenix113 pushed a commit that referenced this pull request Jan 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants