Rename script #848
Open
Rename script #848
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
As suggested in Gp2mv3#835 Currently, this script supports the following: - Change of quadri - Change of course code (either the option like SINF or the number like 1252) - Change of course name (like coo, os...) - Change of course title In order to make it work and keep the other scripts functional, I had to change a bit the config.yml, so that it stays with the "names" at the end of file even after PyYAML's reordering. Current limitations: - Has some difficulties when different courses share the same course name, as it considers that only one course has a given course name (in database parlance, the course name is a primary key). This breaks for crypto q7/q8, and also for meca, elec, math etc courses. - For now, exams and tests have their \hypertitle also updated. Maybe we should keep the old quadri/title/course code in old exams. Open to discussion (it's a 3 lines fix in the code). - It leaves the working tree without any addition to Git; maybe we can integrate Git with it, so that it git adds the files. - On Windows, one has to call it using explicitly python; it's more a feature than a bug however... Bad things: - It's extremely over-engineered probably, but at least it supports nearly all corner cases that I can think of. - Python 3 + Windows + Unicode + PyYAML = nuclear explosion of bugs and strange issues.
|
Shouldn't limitations be considered "bad things" as well 😉? I'll look at this when I have some time. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As suggested in #835
Currently, this script supports the following:
In order to make it work and keep the other scripts functional, I had to change a bit the config.yml, so that it stays with the "names" at the end of file even after PyYAML's reordering.
Current limitations:
Bad things: