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

refactor: Make compatible with neo4j-migrations. #6

Merged
merged 1 commit into from
Feb 21, 2022
Merged

refactor: Make compatible with neo4j-migrations. #6

merged 1 commit into from
Feb 21, 2022

Conversation

michael-simons
Copy link
Contributor

In essence, the whole batch of changes included:

  • Somewhat standardized migrations name (V1_2_3__name.cypher)
  • Replaces id with version in the migration nodes
  • Do a path matching instead of <= id
  • Allows for arbitrary named verions
  • Uses an actual date in the time of migrations
  • Recording of the duration a single migration took
  • Use the same checksum algorithm as neo4j-migrations

I locally ensured that morpheues can read neo4j-migrations and vice
versa:

image

In essence, the whole batch of changes included:

- Somewhat standardized migrations name (`V1_2_3__name.cypher`)
- Replaces `id` with `version` in the migration nodes
- Do a path matching instead of <= id
- Allows for arbitrary named verions
- Uses an actual date in the time of migrations
- Recording of the duration a single migration took
- Use the same checksum algorithm as neo4j-migrations

I locally ensured that morpheues can read neo4j-migrations and vice
versa.
@marianozunino marianozunino merged commit 23935af into marianozunino:develop Feb 21, 2022
@marianozunino
Copy link
Owner

Thank you so much for your contribution!

@marianozunino marianozunino mentioned this pull request Feb 21, 2022
@marianozunino
Copy link
Owner

🎉 This PR is included in version 1.0.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@marianozunino marianozunino mentioned this pull request Feb 21, 2022
marianozunino added a commit that referenced this pull request Feb 21, 2022
* chore: add eslint rule

* refactor: create repository

Created a repository to facilitate testing

* chore: add build status badge

* test(repository): wip

* refactor: move tests files to test dir

* fix: test cov output directory

* test: unit test neo4j

* refactor!: Make compatible with neo4j-migrations. (#6)

In essence, the whole batch of changes included:

- Somewhat standardized migrations name (`V1_2_3__name.cypher`)
- Replaces `id` with `version` in the migration nodes
- Do a path matching instead of <= id
- Allows for arbitrary named verions
- Uses an actual date in the time of migrations
- Recording of the duration a single migration took
- Use the same checksum algorithm as neo4j-migrations

* refactor: rename type Neo4jMigration to Neo4jMigrationNode

* test: increment utils coverage

* test: increment repository coverage

* chore: update readme

* chore: update dev dependencies

Co-authored-by: Michael Simons <michael@simons.ac>
marianozunino added a commit that referenced this pull request Feb 21, 2022
* chore: add eslint rule

* refactor: create repository

Created a repository to facilitate testing

* chore: add build status badge

* test(repository): wip

* refactor: move tests files to test dir

* fix: test cov output directory

* test: unit test neo4j

* refactor: Make compatible with neo4j-migrations. (#6)

BREAKING CHANGE: In essence, the whole batch of changes included:

- Somewhat standardized migrations name (`V1_2_3__name.cypher`)
- Replaces `id` with `version` in the migration nodes
- Do a path matching instead of <= id
- Allows for arbitrary named verions
- Uses an actual date in the time of migrations
- Recording of the duration a single migration took
- Use the same checksum algorithm as neo4j-migrations

* refactor: rename type Neo4jMigration to Neo4jMigrationNode

* test: increment utils coverage

* test: increment repository coverage

* chore: update readme

* chore: update dev dependencies

Co-authored-by: Michael Simons <michael@simons.ac>
marianozunino added a commit that referenced this pull request Feb 21, 2022
* chore: add eslint rule

* refactor: create repository

Created a repository to facilitate testing

* chore: add build status badge

* test(repository): wip

* refactor: move tests files to test dir

* fix: test cov output directory

* test: unit test neo4j

* refactor: Make compatible with neo4j-migrations. (#6)

BREAKING CHANGE: In essence, the whole batch of changes included:

- Somewhat standardized migrations name (`V1_2_3__name.cypher`)
- Replaces `id` with `version` in the migration nodes
- Do a path matching instead of <= id
- Allows for arbitrary named verions
- Uses an actual date in the time of migrations
- Recording of the duration a single migration took
- Use the same checksum algorithm as neo4j-migrations

* refactor: rename type Neo4jMigration to Neo4jMigrationNode

* test: increment utils coverage

* test: increment repository coverage

* chore: update readme

* chore: update dev dependencies

Co-authored-by: Michael Simons <michael@simons.ac>
github-actions bot pushed a commit that referenced this pull request Feb 21, 2022
# [2.0.0](v1.0.0...v2.0.0) (2022-02-21)

### Features

* release ([#9](#9)) ([b63e988](b63e988)), closes [#6](#6)

### BREAKING CHANGES

* In essence, the whole batch of changes included:

- Somewhat standardized migrations name (`V1_2_3__name.cypher`)
- Replaces `id` with `version` in the migration nodes
- Do a path matching instead of <= id
- Allows for arbitrary named verions
- Uses an actual date in the time of migrations
- Recording of the duration a single migration took
- Use the same checksum algorithm as neo4j-migrations

* refactor: rename type Neo4jMigration to Neo4jMigrationNode

* test: increment utils coverage

* test: increment repository coverage

* chore: update readme

* chore: update dev dependencies

Co-authored-by: Michael Simons <michael@simons.ac>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants