-
Notifications
You must be signed in to change notification settings - Fork 9
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
Apply template to packages and gather feedback #106
Comments
Hello, I have tried to apply CopierTemplate.jl to one of my projects and I have the following comments:
Overwrite docs/make.jl? [Y/n] It would be nice if you could see the diff, maybe resolve the conflicts like in git?
|
Hi @luisaforozco, thanks for the feedback. Here is some follow through:
Thanks! |
Created TulipaEnergy/TulipaIO.jl#50 and TulipaEnergy/TulipaIO.jl#51, which are part of the NextGenOpt project.
README.md and docs/src/index.md are mostly lost in the first application or the first change in all contributors.
In this case, the Julia codes were using 4 spaces for indentation and most other files were using 2, so it made several changes. |
Choose a package, apply the template, see the result, gather feedback.
Some possible questions:
pre-commit run -a
changes the existing package structure?Warning
Don't create pull request adding the changes to the package without discussing it in an issue with the package owners first.
The text was updated successfully, but these errors were encountered: