-
Notifications
You must be signed in to change notification settings - Fork 59
Dev r85 v2 pull #538
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
Draft
varmar05
wants to merge
48
commits into
develop
Choose a base branch
from
dev-r85-v2-pull
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Dev r85 v2 pull #538
Conversation
This file contains hidden or 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
Expand diff json column into separate table
Introduce file diff table
Specification changed - rollback recent changes
Merged diffs
- introduce get_delta method with caching - introduce delta contrller
- added tests for pull and checkpoints
- accept numbers in endpoint - consider to store diff instead of diffs in db
- rename table to project version delta with column changes - rename classes - add get_delta_changes to project instance (nice) - fix migration to int
Fix alembic migration for file diff
- update new table schema for project version delta - add tests and upgrade existing to handle order of diffs
- handle delete project cleanup of delta checkpoints
- update reponse to items: [] - make it more clear with changes logic (?) - @varmar05
Checkpoints introduction for version changes
API: add 'v' prefix to version in delta endpoint
- for file diffs we only create higher ranks if it makes sense (contains changes) - if delta cannot be created / fetched return on error
Do not update global config variable for gevent mode. Make sure we do not use gevent env for tests apart of dedicated tests. In those tests mock configuration rather than modifing global variable.
Create checkpoint recursively
Pull Request Test Coverage Report for Build 19732530660Details
💛 - Coveralls |
Publish v2 pull enabled flag
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.
No description provided.