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

CI: Add edb label to labeler #3761

Merged
merged 1 commit into from
Oct 17, 2023
Merged

CI: Add edb label to labeler #3761

merged 1 commit into from
Oct 17, 2023

Conversation

SMoraisAnsys
Copy link
Collaborator

Update the Github workflow labeler to track changes related to edb (based on specific modified files).
This could me removed later on but would be of great help for the split between AEDT and EDB.

@ansys-reviewer-bot
Copy link
Contributor

Thanks for opening a Pull Request. If you want to perform a review write a comment saying:

@ansys-reviewer-bot review

@SMoraisAnsys
Copy link
Collaborator Author

@ansys-reviewer-bot review

@ansys-reviewer-bot
Copy link
Contributor

Okay, I will trigger a review of your PR.

- examples/01-HFSS3DLayout/EDB_in_3DLayout.py
- examples/05-Q3D/Q3D_from_EDB.py
- pyaedt/edb_core/**
- pyaedt/edb.py
Copy link
Contributor

Choose a reason for hiding this comment

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

Always include a newline at the end of the file. To fix this, add an empty line at the end of the labeler.yml file:

Suggested change
- pyaedt/edb.py
+- pyaedt/edb.py

Other than the newline at the end of the file, the patch seems fine, and adds new EDB categories for the labeler.

@github-actions github-actions bot added the maintenance Package and maintenance related label Oct 17, 2023
@codecov
Copy link

codecov bot commented Oct 17, 2023

Codecov Report

Merging #3761 (c4614cb) into main (4f19a93) will increase coverage by 49.63%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #3761       +/-   ##
===========================================
+ Coverage   31.43%   81.07%   +49.63%     
===========================================
  Files         179      179               
  Lines       61935    61935               
===========================================
+ Hits        19468    50211    +30743     
+ Misses      42467    11724    -30743     

Copy link
Member

@Samuelopez-ansys Samuelopez-ansys left a comment

Choose a reason for hiding this comment

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

LGTM

@SMoraisAnsys SMoraisAnsys merged commit a75ce98 into main Oct 17, 2023
12 checks passed
@SMoraisAnsys SMoraisAnsys deleted the ci/update_labeler branch October 17, 2023 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Package and maintenance related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants