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 from pipenv to poetry #240

Closed
wants to merge 30 commits into from
Closed

Conversation

alichtman
Copy link
Owner

@alichtman alichtman commented Nov 14, 2019

Fix #240

@alichtman alichtman mentioned this pull request Nov 14, 2019
alichtman and others added 22 commits March 22, 2020 16:56
* Begin reworking tests

Progress on #237

* Respect XDG Base Directory spec (#239)

Fix #236

* Fix config creation bug

* Get all tests to pass

Fix #237

* Apparently travis VMs don't come with /tmp created already

* Wrong diagnosis of the issue
* Move config to ~/.config/shallow-backup.conf

* Bump version to v3.2
* Fixes #216

* Added Test Coverage for reinstallation of dotfiles regarding #216

* Refactor reinstallation code

Co-authored-by: Supratim <sup270792@gmail.com>
This issue is still unfixed and Pipfile is unusable.
pypa/pipenv#3827 (comment)
- Add root-gitignore and dotfiles-gitignore keys to
config to replace default-gitignore.
- Move all gitignore functionality to config.
- Add tests for new .git-related operations
- Bump version to v4.0
@alichtman
Copy link
Owner Author

Codacy Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 5
- Added 1
           

Complexity increasing per file
==============================
- shallow_backup/upgrade.py  10
         

See the complete overview on Codacy

@@ -29,7 +32,7 @@
@click.option('-separate_dotfiles_repo', is_flag=True, default=False, help="Use if you are trying to maintain a separate dotfiles repo and running into issue #229.")
@click.option('-show', is_flag=True, default=False, help="Display config file.")
@click.option('--version', '-v', is_flag=True, default=False, help='Display version and author info.')
def cli(all, configs, delete_config, destroy_backup, dotfiles, fonts, new_path,
def cli(add_dot, all, configs, delete_config, destroy_backup, dotfiles, fonts, new_path,
Copy link
Owner Author

Choose a reason for hiding this comment

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

@alichtman alichtman closed this Oct 11, 2023
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.

1 participant