Skip to content

[CI] Removed push event from workflows#25

Closed
orensbruli wants to merge 3 commits intomainfrom
emartinena/remove_push_event
Closed

[CI] Removed push event from workflows#25
orensbruli wants to merge 3 commits intomainfrom
emartinena/remove_push_event

Conversation

@orensbruli
Copy link
Copy Markdown
Contributor

Signed-off-by: Esteban Martinena orensbruli@gmail.com

Signed-off-by: Esteban Martinena <orensbruli@gmail.com>
@orensbruli orensbruli force-pushed the emartinena/remove_push_event branch from 5801ef4 to 1d0bfe4 Compare September 21, 2022 14:14
@@ -1,6 +1,5 @@
name: build
on:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe we wanna run the workflow when a merge happens? I'm not sure if the pull_request will run it at merge time, if it doesn't maybe we wanna keep something like this?

Suggested change
on:
on:
push:
branches:
- master

Signed-off-by: Esteban Martinena <orensbruli@gmail.com>
Signed-off-by: Esteban Martinena <orensbruli@gmail.com>
@orensbruli
Copy link
Copy Markdown
Contributor Author

Closing in favor of #24 (changes merged in that branch).

@orensbruli orensbruli closed this Dec 8, 2022
@orensbruli orensbruli deleted the emartinena/remove_push_event branch December 8, 2022 07:59
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.

2 participants