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

docs: documenting new naming conventions for commits, branches and PRs. #3228

Conversation

germa89
Copy link
Collaborator

@germa89 germa89 commented Jun 27, 2024

Description

As the title. I want to have everything clearly stated in the PyMAPDL development section.

Issue linked

Close #3218

Checklist

@germa89 germa89 requested a review from a team as a code owner June 27, 2024 16:35
@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

@github-actions github-actions bot added the documentation Documentation related (improving, adding, etc) label Jun 27, 2024
@germa89 germa89 self-assigned this Jun 27, 2024
@germa89 germa89 requested review from klmcadams and PipKat June 27, 2024 16:37
@germa89 germa89 enabled auto-merge (squash) June 27, 2024 16:37
@germa89 germa89 requested a review from a team June 27, 2024 16:38
@wiz-inc-572fc38784
Copy link
Contributor

wiz-inc-572fc38784 bot commented Jun 27, 2024

Wiz Scan Summary

IaC Misconfigurations 0C 0H 0M 0L 0I
Vulnerabilities 0C 0H 0M 0L 0I
Sensitive Data 0C 0H 0M 0L 2I
Total 0C 0H 0M 0L 2I
Secrets 0🔑

@germa89
Copy link
Collaborator Author

germa89 commented Jun 27, 2024

NOT finished.

I need to fix the labels.

@germa89 germa89 added the DO NOT MERGE Not ready to be merged yet label Jun 27, 2024
@germa89 germa89 marked this pull request as draft June 27, 2024 16:42
auto-merge was automatically disabled June 27, 2024 16:42

Pull request was converted to draft

Copy link

codecov bot commented Jun 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.21%. Comparing base (cda716a) to head (68cb8ef).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3228      +/-   ##
==========================================
- Coverage   82.64%   80.21%   -2.44%     
==========================================
  Files          53       53              
  Lines        9625     9625              
==========================================
- Hits         7955     7721     -234     
- Misses       1670     1904     +234     

@germa89 germa89 requested a review from RobPasMue June 28, 2024 08:18
@germa89 germa89 marked this pull request as ready for review June 28, 2024 08:18
@github-actions github-actions bot added CI/CD Related with CICD, Github Actions, etc maintenance General maintenance of the repo (libraries, cicd, etc) labels Jun 28, 2024
@germa89
Copy link
Collaborator Author

germa89 commented Jun 28, 2024

Ok. Now it is ready.

.github/labels.yml Outdated Show resolved Hide resolved
.github/labels.yml Outdated Show resolved Hide resolved
Copy link
Member

@PipKat PipKat left a comment

Choose a reason for hiding this comment

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

Minor edits--mostly to get rid of "etc."--as we want to avoid Latin phrases.

@germa89 germa89 removed the DO NOT MERGE Not ready to be merged yet label Jun 28, 2024
germa89 and others added 2 commits June 28, 2024 17:43
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
@germa89 germa89 enabled auto-merge (squash) June 28, 2024 15:49
Copy link
Contributor

@MaxJPRey MaxJPRey left a comment

Choose a reason for hiding this comment

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

LGTM.

@germa89
Copy link
Collaborator Author

germa89 commented Jul 1, 2024

many people approved it. Hence @pyansys-ci-bot LGTM.

Copy link
Member

@jorgepiloto jorgepiloto left a comment

Choose a reason for hiding this comment

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

✅ Approving this PR because germa89 said so in here 😬

LGTM

@germa89 germa89 disabled auto-merge July 1, 2024 06:26
@germa89
Copy link
Collaborator Author

germa89 commented Jul 1, 2024

I had to disable Require review from Code Owners because @jorgepiloto is not a code owner.

image

P.D.- @pyansys-ci-bot token has been replaced by a @jorgepiloto's one. Hence for this PR to be merged I had to disable the above requirement, since the workflow now is triggered with @jorgepiloto's credentials which is not a codeowner.

@germa89 germa89 merged commit c386f0c into main Jul 1, 2024
44 checks passed
@germa89 germa89 deleted the docs/document-new-naming-conventions-for-commits,-branches-and-PRs branch July 1, 2024 07:07
@RobPasMue
Copy link
Member

Why not add him to the team temporarily?

@germa89
Copy link
Collaborator Author

germa89 commented Jul 1, 2024

Codeowners require to edit the Codeowners file. I rather change not change files. :)

@RobPasMue
Copy link
Member

RobPasMue commented Jul 1, 2024

No, you do not need to change the file - you just need to add @jorgepiloto to the @ansys/pymapdl-maintainers team.

* @ansys/pymapdl-maintainers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD Related with CICD, Github Actions, etc documentation Documentation related (improving, adding, etc) maintenance General maintenance of the repo (libraries, cicd, etc)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update documentation to changelog changes
8 participants