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

build: change install_poetry call in devtool functions #27

Merged
merged 5 commits into from
Oct 11, 2023

Conversation

Satish615
Copy link
Contributor

Description of changes:
change install_poetry call in devtool functions

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

pinaraws
pinaraws previously approved these changes Oct 11, 2023
xiaoyi-cheng
xiaoyi-cheng previously approved these changes Oct 11, 2023
pinaraws
pinaraws previously approved these changes Oct 11, 2023
@xiaoyi-cheng
Copy link
Contributor

I clicked on "Update branch" but both checks shows "expected", actions shows failed -- actions should be triggered when we update branch on the page.

keerthanvasist
keerthanvasist previously approved these changes Oct 11, 2023
@Satish615
Copy link
Contributor Author

Satish615 commented Oct 11, 2023

I clicked on "Update branch" but both checks shows "expected", actions shows failed -- actions should be triggered when we update branch on the page.

I am not able to replicate duplicate builds for commit push on my pull request for different events.

  1. push commit before pr is raised.
  2. push commit after pr is raised.
  3. force push commit after pr is raised.

So I will leave the trigger events as is for now.

@Satish615 Satish615 merged commit e133fbb into aws:main Oct 11, 2023
2 checks passed
@Satish615 Satish615 deleted the poetry_test branch October 11, 2023 14:04
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