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

Transition to running action in JS instead of Docker #22

Merged
merged 2 commits into from
Jan 26, 2021

Conversation

davidparsson
Copy link
Contributor

Fixes #21, hopefully.

It's heavily inspired by pascalgn/automerge-action#125 and I haven't been able to test it locally.

@davidparsson davidparsson force-pushed the convert-to-js-action branch 9 times, most recently from 3fad121 to a456717 Compare January 22, 2021 08:35
@davidparsson
Copy link
Contributor Author

davidparsson commented Jan 22, 2021

I have made a test run here, and it seems to be working: davidparsson#2

I created another branch in which I defined that davidparsson/size-label-action@convert-to-js-action (the source branch of this PR) should be used as an action. I then created a PR to my own repo for that branch.

@pascalgn pascalgn merged commit 1bdcc27 into pascalgn:master Jan 26, 2021
@pascalgn
Copy link
Owner

Sorry for the delay and thanks for the PR, it's merged now! 👍

@davidparsson davidparsson deleted the convert-to-js-action branch January 27, 2021 19:04
tetienne added a commit to livestorm/size-label-action that referenced this pull request Feb 24, 2021
* Use pull request base as home

When creating a PR from a fork it could previously not find the PR (pascalgn#9)

* Fixed tests

* Update dependencies

* Add action.yml

* Update README.md

* Update dependencies

* Update dependencies

* Update README.md

* Update dependencies

* Update README.md

* Update README.md (pascalgn#14)

* Update README.md

* Update dependencies

* Update README.md

* Support reopen PR action (pascalgn#18)

* Update README.md

* Transition to running action in JS instead of Docker (pascalgn#22)

* Update README.md

* feat: do not fail on label remove error (pascalgn#19)

* Update README.md

* Update dist/index.js

* Update README.md

Co-authored-by: Daniel Schmidt <[email protected]>
Co-authored-by: Pascal <[email protected]>
Co-authored-by: Pascal <[email protected]>
Co-authored-by: Carlos Alexandro Becker <[email protected]>
Co-authored-by: David Pärsson <[email protected]>
Co-authored-by: Zadkiel <[email protected]>
tetienne added a commit to livestorm/size-label-action that referenced this pull request Nov 16, 2021
* Use pull request base as home

When creating a PR from a fork it could previously not find the PR (pascalgn#9)

* Fixed tests

* Update dependencies

* Add action.yml

* Update README.md

* Update dependencies

* Update dependencies

* Update README.md

* Update dependencies

* Update README.md

* Update README.md (pascalgn#14)

* Update README.md

* Update dependencies

* Update README.md

* Support reopen PR action (pascalgn#18)

* Update README.md

* Transition to running action in JS instead of Docker (pascalgn#22)

* Update README.md

* feat: do not fail on label remove error (pascalgn#19)

* Update README.md

* Update dist/index.js

* Update README.md

Co-authored-by: Daniel Schmidt <[email protected]>
Co-authored-by: Pascal <[email protected]>
Co-authored-by: Pascal <[email protected]>
Co-authored-by: Carlos Alexandro Becker <[email protected]>
Co-authored-by: David Pärsson <[email protected]>
Co-authored-by: Zadkiel <[email protected]>
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.

Convert to pure JavaScript Action?
2 participants