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

Removes formatting feature #29

Merged
merged 13 commits into from
Jan 10, 2021
Merged

Removes formatting feature #29

merged 13 commits into from
Jan 10, 2021

Conversation

rickstaa
Copy link
Member

@rickstaa rickstaa commented Jan 7, 2021

After a discussion with @haya14busa we decided to remove the formatting action again as this should be left inside a seperate repository (see reviewdog/action-remark-lint#25). This repository can be found on https://github.com/rickstaa/action-black.

After a discussion with @haya14busa we decided to remove the formatting
action again as this should be left inside a seperate repository
(see reviewdog/action-remark-lint#25). This
repository can be found on https://github.com/rickstaa/action-black.
@rickstaa rickstaa changed the title 🔥 Removes formatting feature Removes formatting feature Jan 7, 2021
@rickstaa
Copy link
Member Author

rickstaa commented Jan 7, 2021

@haya14busa What do you think should I add a minor or major version change label for this?

@haya14busa
Copy link
Member

@haya14busa What do you think should I add a minor or major version change label for this?

Maybe we can bump major version because it's breaking change.
I'm fine to just bump minor/patch version if it's just released recently and the # of users is small.
I leave it to you.

@rickstaa rickstaa force-pushed the remove_formatting_feature branch from 9d64a30 to 94e0d79 Compare January 8, 2021 13:03
@rickstaa
Copy link
Member Author

rickstaa commented Jan 8, 2021

🧪 Additional (successful) test results can be found rickstaa/action-test-repo#51.

@tsuyoshicho
Copy link
Contributor

alex disable specifc word

<!--alex disable black-->

Add to entrypoint.sh for clearing check.

see get-alex/alex - Control

This commit changes the `black_flags` input argument to `black_args` as
this better describes what it does. Additionally it contains some small
bug fixes.
@rickstaa rickstaa requested a review from haya14busa January 10, 2021 09:51
@rickstaa rickstaa added the bump:major Bump the major version on pull request label Jan 10, 2021
@github-actions
Copy link
Contributor

🏷️ [bumpr] Next version:v2.0.0 Changes:v1.4.0...reviewdog:remove_formatting_feature

@rickstaa rickstaa merged commit 1e5ee4c into master Jan 10, 2021
@github-actions
Copy link
Contributor

🚀 [bumpr] Bumped! New version:v2.0.0 Changes:v1.4.0...v2.0.0

@rickstaa rickstaa deleted the remove_formatting_feature branch January 10, 2021 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump:major Bump the major version on pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants