Skip to content

ci: fix release script - #54

Merged
y-lakhdar merged 12 commits into
mainfrom
fix-release-script
Jun 13, 2023
Merged

y-lakhdar merged 12 commits into
mainfrom
fix-release-script

Conversation

@y-lakhdar

Copy link
Copy Markdown
Contributor

The following PR #53 was not automatically merged by developer-experience-bot.
I think the condition was invalid, which skipped the workflow

@y-lakhdar
y-lakhdar requested review from a team, btaillon-coveo, louis-bompart and olamothe and removed request for a team June 13, 2023 14:11
@github-actions

github-actions Bot commented Jun 13, 2023

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Manifest Files

@y-lakhdar

Copy link
Copy Markdown
Contributor Author

I just need another pair of eyes to ensure that's the right condition.
Summoning @louis-bompart, the GH workflow master

@louis-bompart

Copy link
Copy Markdown
Contributor

You should be able to query the GH API to list all workflows run attempts, and see who triggered it (actor name included)

Can you double-check it?

@y-lakhdar

Copy link
Copy Markdown
Contributor Author

image
Turns out the actor value was correct

@y-lakhdar
y-lakhdar enabled auto-merge (squash) June 13, 2023 19:32
@y-lakhdar
y-lakhdar merged commit 57dc725 into main Jun 13, 2023
@y-lakhdar
y-lakhdar deleted the fix-release-script branch June 13, 2023 19:33
y-lakhdar added a commit that referenced this pull request Jun 20, 2023
* ci: use expression syntax
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.

3 participants