Skip to content

Improvements#4

Merged
Marc-André Moreau (awakecoding) merged 13 commits intomasterfrom
improvements
Feb 28, 2019
Merged

Improvements#4
Marc-André Moreau (awakecoding) merged 13 commits intomasterfrom
improvements

Conversation

@thenextman
Copy link
Copy Markdown
Member

@thenextman Richard Markiewicz (thenextman) commented Feb 28, 2019

  • You can control the verbosity with v (e.g. -vvv)
  • Better behaviour when the user is using an existing source control repo (we try to fetch the latest version, but error out if this would cause conflicts)
  • Standardize the output of the vcs-log command:

`%commit% - %date% - %author% - %message%

Where commit is the short hash in git, and date is ISO formatted

  • Add the ability to specify the target language(s)
  • Pulling "updated" strings is now optional (with --show-changes). Mixed-language files are never created.
  • The force option is no longer available when pushing; strings that have not been updated from the source language are simply not merged
  • Filenames now contain the revision(s) that the operation was performed on. So for example, this command:

cirup pull --old-commit r123 --new-commit r456

Will produce translation files named like this:

UIResources.fr.~123-456~.resx

The file can be updated, then merged back with the corresponding push:

cirup push --old-commit r123 --new-commit r456

  • Bug fixes, crashes and formatting!

- general output improvements
- better behaviour with users local repository
- standardised vcs formatting output
- specify target languages
- no longer allow  push changes
- add  flag when pulling
- change merge query semantics
- pull and push insert revision ranges into filenames (WIP)
- documentation (WIP)
- move revision struct into seperate file
- fix handling of revision markers in filenames
- fix visibility of modules
- debug print shell commands
- add a limit parameter to vcs-log
- standardise date format in vcs-log
- use vec instead of hashmap to hold languages
- fix merge query
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants