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

Restructure Examples folder #382

Closed
wants to merge 0 commits into from
Closed

Conversation

sebsto
Copy link
Contributor

@sebsto sebsto commented Sep 30, 2024

Move all v1 examples to folder
Examples/v1
Keep only v2 examples in Examples folder

@sebsto sebsto requested a review from fabianfett September 30, 2024 13:51
Copy link
Member

@fabianfett fabianfett left a comment

Choose a reason for hiding this comment

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

First round

Examples/APIGateway/Sources/main.swift Outdated Show resolved Hide resolved
Examples/APIGateway/Sources/main.swift Outdated Show resolved Hide resolved
@@ -0,0 +1,53 @@
// swift-tools-version:5.7

import PackageDescription
Copy link
Member

Choose a reason for hiding this comment

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

Why do we want to keep the v1 examples?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

For history and retro compatibility.

@sebsto
Copy link
Contributor Author

sebsto commented Sep 30, 2024

Done. @fabianfett do you have a hard feeling against keeping v1's ?

@fabianfett
Copy link
Member

Done. @fabianfett do you have a hard feeling against keeping v1's ?

Kinda. When users look at the repo, they don't necessarily know which version we are currently at. This might lead to confusion. Examples for v1 are still available in the v1 branch.

@sebsto
Copy link
Contributor Author

sebsto commented Oct 7, 2024

@fabianfett There is no 1.0.0 branches, these are just tags.
I propose to create a branch from the last 1.0.0 tag and delete everything v1 specific from main. OK ?

@sebsto sebsto closed this Oct 8, 2024
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