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

refactor: Restructure repo #98

Merged
merged 10 commits into from
May 15, 2024
Merged

refactor: Restructure repo #98

merged 10 commits into from
May 15, 2024

Conversation

gintsk
Copy link
Contributor

@gintsk gintsk commented Feb 18, 2024

Contributes to #96

  • Move MimeMapping project to src/MimeMapping
  • Rename Test to MimeMapping.Tests and move test project to test/MimeMapping.Tests
  • use the new artifacts output
  • use global.json with .NET SDK version 8.0.100 (8.0.200 not yet available for builds) will do it in a separate PR

@viceice please check .github/workflows as changes may be required due to use of artifacts output.

Copy link
Collaborator

@viceice viceice left a comment

Choose a reason for hiding this comment

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

yes, needs workflow changes too

@gintsk
Copy link
Contributor Author

gintsk commented Mar 2, 2024

@viceice any chances you could look into necessary workflow changes so we could finish this PR?

@viceice viceice changed the title Restructurate repo refactor: Restructurate repo Mar 2, 2024
Copy link
Collaborator

@viceice viceice left a comment

Choose a reason for hiding this comment

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

move Test project to test/MimeMapping.Tests

Directory.Build.props Outdated Show resolved Hide resolved
global.json Outdated Show resolved Hide resolved
@viceice
Copy link
Collaborator

viceice commented Mar 2, 2024

will look at required workflow changes the next days.

MimeMapping.sln Outdated Show resolved Hide resolved
src/MimeMapping/MimeMapping.snk Outdated Show resolved Hide resolved
test/MimeMapping.Tests/test.runsettings Outdated Show resolved Hide resolved
@gintsk
Copy link
Contributor Author

gintsk commented Mar 27, 2024

This PR seems stalled. Should I close it?

@viceice
Copy link
Collaborator

viceice commented Apr 10, 2024

This PR seems stalled. Should I close it?

Why? you need to re-request review when you've done all required changes
image

Please also fix the conflicts

@gintsk
Copy link
Contributor Author

gintsk commented May 3, 2024

It seems all previous issues are addressed and fixed. Except for any potentially required workflow changes - which you wrote you will tackle.

@gintsk gintsk requested a review from viceice May 3, 2024 19:28
.github/workflows/build.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
src/MimeMapping/MimeMapping.csproj Outdated Show resolved Hide resolved
test/MimeMapping.Tests/MimeMapping.Tests.csproj Outdated Show resolved Hide resolved
@gintsk gintsk requested a review from viceice May 5, 2024 15:02
Copy link
Collaborator

@viceice viceice left a comment

Choose a reason for hiding this comment

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

the mentioned changes should not be visible in this PR. you probably did some wrong merge / rebase. please fix this and force push if required

@viceice
Copy link
Collaborator

viceice commented May 5, 2024

ok, will fix your PR tomorrow

@gintsk gintsk closed this May 15, 2024
@viceice
Copy link
Collaborator

viceice commented May 15, 2024

@gintsk why closed?

@viceice viceice reopened this May 15, 2024
@viceice viceice changed the title refactor: Restructurate repo refactor: Restructure repo May 15, 2024
@viceice viceice enabled auto-merge (squash) May 15, 2024 12:26
@viceice viceice mentioned this pull request May 15, 2024
4 tasks
@viceice viceice merged commit 5d54f34 into zone117x:master May 15, 2024
4 checks passed
@gintsk gintsk deleted the restructurate_repo branch May 15, 2024 13:49
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.

2 participants