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

move cgi to nimble packages #20951

Closed
wants to merge 4 commits into from
Closed

move cgi to nimble packages #20951

wants to merge 4 commits into from

Conversation

ringabout
Copy link
Member

@ringabout ringabout commented Nov 28, 2022

https://github.com/nim-lang/cgi

I will start with this simple package, and apply the process to other packages.

@ringabout
Copy link
Member Author

ringabout commented Nov 28, 2022

I decided to remove cgi instead of giving an error because the former solution doesn't require users to change the code after installing cgi. The latter solution enforces using pkg/cgi. We change the name though.

@ringabout ringabout changed the title move cgi to nimble packages [Experiment] move cgi to nimble packages Nov 28, 2022
@ringabout
Copy link
Member Author

For karax karaxnim/karax#233

@ringabout ringabout changed the title [Experiment] move cgi to nimble packages move cgi to nimble packages Nov 28, 2022
@ringabout
Copy link
Member Author

for rosencrantz andreaferretti/rosencrantz#30

@Varriount Varriount requested a review from Araq November 28, 2022 20:17
@Varriount Varriount added the Requires Araq To Merge PR should only be merged by Araq label Nov 28, 2022
@metagn
Copy link
Collaborator

metagn commented Nov 29, 2022

Also remove from doc/lib.md (shouldn't cause conflicts)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Requires Araq To Merge PR should only be merged by Araq
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants