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

Rework the github workflow #814

Merged
merged 32 commits into from
Aug 2, 2023
Merged

Conversation

martijn00
Copy link
Contributor

@martijn00 martijn00 commented Aug 1, 2023

@ebozduman

According to good practice and latest standards there are some things that should change. You need to create a develop and a main branch. The default branch should be develop.

Also a NUGET_API_KEY needs to be added to the secrets

@martijn00
Copy link
Contributor Author

@ebozduman can you check this and run?

@ebozduman
Copy link
Collaborator

@martijn00

I need to ask permission/blessing of the team as they might also have some kind of a plan already in place.
In addition, I don't have permission to make the change.
Renaming master is going to be mandatory in October for all in github.com.
I let them know. I'll hear from them tomorrow (actually today in 6-8 hours).

I kicked of all the checks but of course, nothing happens as the workflows depend on the new main and develop branches to exist.
So, waiting for the Team's blessing and action to replace master, and create a develop branch.

@martijn00
Copy link
Contributor Author

@ebozduman I've added master to the script now. This should make it possible to merge now, and wait for the team to agree on the develop and main change.

@martijn00
Copy link
Contributor Author

@ebozduman can you run again?

@ebozduman
Copy link
Collaborator

@martijn00

It is stuck again !?!? Is the added master branch gone somehow?

Meanwhile,
Not now but later, we can also add one of the old officially supported .net framework versions by MicroSoft ,

4.6.1, 4.6.2, 4.7, 4.7.1, 4.7.2, 4.8, 4.8.1

let's say 4.7.2 or 4.8.1, as a legacy net framework sample build check.

@martijn00
Copy link
Contributor Author

Yeah we can add a framework like that later. The build is not stuck, you just need to run the last commit.

@ebozduman
Copy link
Collaborator

Started.
It did not display the check Start button first, but now I see it.

@martijn00
Copy link
Contributor Author

Look at the flow now: https://github.com/minio/minio-dotnet/actions/runs/5738231383

Looks nice with jobs splitting up! Later on I could maybe even bring in more improvements.

@martijn00
Copy link
Contributor Author

@ebozduman it passed the build! Can you merge now?

@ebozduman
Copy link
Collaborator

Running again

@ebozduman
Copy link
Collaborator

By the way, I also noticed the entry end_of_line = crlf rule in .editorConfig file.
That would trip your PR #815 check.
I am pretty sure, I replaced it with lf, but I don't see any records of it nowhere in github, and #802 also did not have it.
I'll create a PR for it.

Copy link
Collaborator

@ebozduman ebozduman left a comment

Choose a reason for hiding this comment

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

LGTM

@ebozduman
Copy link
Collaborator

@martijn00

The build check is waiting for status to be reported for a 6.0 build, which is supposed to be not required anymore.

@martijn00
Copy link
Contributor Author

How can that be removed? It is not in the new script anymore

@ebozduman
Copy link
Collaborator

I am also not sure how. Checking...

@martijn00
Copy link
Contributor Author

@ebozduman I found this:

The status "Expected — Waiting for status to be reported" typically appears when a GitHub Actions workflow is expected to run on a commit or pull request, but has not yet reported a status. If the workflow was removed or renamed in the repository, this status might be left "hanging" and the check might become a "stale check."

To remove a stale check, you need to update the branch protection rules, if you have the necessary permissions (i.e., you are an owner or administrator of the repository). Here's how to do it:

Go to your GitHub repository.
Click on "Settings" on the right side of the repository's page.
Click on "Branches" in the left sidebar.
Under the "Branch protection rules" section, click on the "Edit" button next to the rule that applies to the branch you're concerned about.
In the "Protect matching branches" section, scroll down to the "Require status checks to pass before merging" area.
You should see a list of status checks found in the last week for this repository. If the stale check is listed here, uncheck it.
Click on "Save changes" at the bottom of the page.
If the stale check isn't listed, it could be that it's not a check from the last week. In this case, you may have to temporarily re-add the check (for example, by restoring the workflow that produced it), push a commit to trigger it, then remove it from your branch protection rules.

Note that only users with admin or owner permissions for the repository can change the branch protection rules. If you don't have the necessary permissions, you'll need to contact someone who does to make the change for you.

@ebozduman
Copy link
Collaborator

Ok. Great!
I need to ask someone who has permission to do that.
Will do.

@harshavardhana harshavardhana merged commit f30c721 into minio:master Aug 2, 2023
7 checks passed
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