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

MAINT: update github workflows files #405

Merged

Conversation

Gui-FernandesBR
Copy link
Member

Pull request type

Please check the type of change your PR introduces:

  • ReadMe, Docs and GitHub maintenance

Pull request checklist

Please check if your PR fulfills the following requirements, depending on the type of PR:

  • ReadMe, Docs and GitHub maintenance:

    • Spelling has been verified
    • Code docs are working correctly

What is the current behavior?

The github workflows are not perfect.

What is the new behavior?

They continue not being perfect, but at least now they are updated. Also:

  • no longer assigns the issue to its author (only for PRs)
  • deleted the useless auto-assign-projects file
  • runs the pytest without the [all] option
  • assigns @phmbressan as a reviewer in the PRs. I was really missing this.

Does this introduce a breaking change?

  • No

Other information

  • This PR already works as a test for the current file
  • Maybe we could merge master into beta/v1 to catch this little guy: 18ceea1

- no longer assigns the issue to its author (only for PRs)
- deleted the useless auto-assign-projects file
- runs the pytest without the [all] option
@Gui-FernandesBR Gui-FernandesBR added this to the Release v1.1.0 milestone Sep 7, 2023
@phmbressan
Copy link
Collaborator

Is this PR really working as intended? You mentioned in the description that it is by itself a test of the new files, but I have not been assigned as a reviewer.

Furthermore, shouldn't the destination branch of this PR be beta/v1.0.0?

.github/workflows/test_pytest.yaml Outdated Show resolved Hide resolved
@Gui-FernandesBR Gui-FernandesBR added the C.I. Continuous Integration (Workflows and actions) label Sep 10, 2023
@Gui-FernandesBR
Copy link
Member Author

Is this PR really working as intended? You mentioned in the description that it is by itself a test of the new files, but I have not been assigned as a reviewer.

Furthermore, shouldn't the destination branch of this PR be beta/v1.0.0?

You know, since there's no way to test it, sometimes you have to trust that the workflows are gonna work 😅😅😅

  • But yes, I can confirm that the same behavior [of you not being assigned as a reviewer yet] happened in the first time we introduced such tests: MAINT: auto assign workflows #214
  • Not actually, the maint/update-git-actions was created from the fix/env-analysis-dependency-imports branch, so if I change the direction to beta/v1 it is gonna show way more additions than what is suppose to show. Maybe a good idea is to merge FIX: env analysis dependency imports #403 before this one!

Anyway, many thanks for reviewing it :)
I'm sending a re-review request now

Copy link
Collaborator

@phmbressan phmbressan left a comment

Choose a reason for hiding this comment

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

I agree with the changes made. Approved.

@Gui-FernandesBR
Copy link
Member Author

Merging it into fix/env-analysis-dependency-imports

@Gui-FernandesBR Gui-FernandesBR merged commit cfdd041 into fix/env-analysis-dependency-imports Sep 13, 2023
8 checks passed
@Gui-FernandesBR Gui-FernandesBR deleted the maint/update-git-actions branch September 14, 2023 18:55
@Gui-FernandesBR
Copy link
Member Author

Is this PR really working as intended? You mentioned in the description that it is by itself a test of the new files, but I have not been assigned as a reviewer.
Furthermore, shouldn't the destination branch of this PR be beta/v1.0.0?

You know, since there's no way to test it, sometimes you have to trust that the workflows are gonna work 😅😅😅

  • But yes, I can confirm that the same behavior [of you not being assigned as a reviewer yet] happened in the first time we introduced such tests: MAINT: auto assign workflows #214
  • Not actually, the maint/update-git-actions was created from the fix/env-analysis-dependency-imports branch, so if I change the direction to beta/v1 it is gonna show way more additions than what is suppose to show. Maybe a good idea is to merge FIX: env analysis dependency imports #403 before this one!

Anyway, many thanks for reviewing it :) I'm sending a re-review request now

Actually, I think the workflow you start working correctly only when it gets to the master branch.
Let's wait a bit more to see.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C.I. Continuous Integration (Workflows and actions)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants