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

atlasaction: add code suggestion comments to migrate lint action #119

Merged
merged 18 commits into from
Feb 19, 2024

Conversation

ronenlu
Copy link
Member

@ronenlu ronenlu commented Feb 14, 2024

see:

image

@ronenlu ronenlu changed the title atlasaction: add code suggestion comment in migrate lint atlasaction: add code suggestion comments to migrate lint action Feb 14, 2024
Copy link

Atlas Lint Report

Analyzed migrations

StatusStepResultDiagnostics
Detect New Migration FilesFound 0 new migration files (from 0 total)
Replay Migration FilesLoaded 0 changes on dev database
Connect your project to Atlas Cloud to get more safety checks

Migrations automatically reviewed by Atlas

Copy link

Atlas Lint Report

Analyzed migrations2

StatusStepResultDiagnostics
Detect New Migration FilesFound 0 new migration files (from 0 total)
Replay Migration FilesLoaded 0 changes on dev database
Connect your project to Atlas Cloud to get more safety checks

Migrations automatically reviewed by Atlas

Copy link

github-actions bot commented Feb 14, 2024

atlas migrate lint on atlasaction/testdata/migrations

Status Step Link
1 new migration file detected  
ERD and Visual Diff generated View Visualization
No issues found Full Report

Migrations automatically reviewed with Atlas

Copy link

github-actions bot commented Feb 14, 2024

atlas migrate lint on atlasaction/testdata/migrations_destructive

Status Step Link
1 new migration file detected  
ERD and Visual Diff generated View Visualization
1 error found Full Report

Migrations automatically reviewed with Atlas

@ronenlu ronenlu force-pushed the code_suggestions branch 6 times, most recently from 5e0c4f1 to 8ff887a Compare February 15, 2024 09:19
@@ -355,20 +355,20 @@ func TestMigrateLint(t *testing.T) {
require.NoError(t, err)
require.Equal(t, 0, len(out))
})
t.Run("lint with error", func(t *testing.T) {
Copy link
Member Author

Choose a reason for hiding this comment

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

this test is included in the test case bellow: lint summary - lint error, thus I deleted it

@ronenlu ronenlu closed this Feb 18, 2024
@ronenlu ronenlu reopened this Feb 18, 2024
@ronenlu ronenlu requested a review from a8m February 18, 2024 15:35
@ronenlu ronenlu merged commit 05df1da into master Feb 19, 2024
7 checks passed
@a8m a8m deleted the code_suggestions branch February 19, 2024 12:58
ronenlu added a commit that referenced this pull request Feb 19, 2024
* update atlas-go-sdk

* small refactor of code

* add code suggestion function

* test commit

* fix test

* update tests

* update atlas go sdk verison

* temp

* update atlas sdk

* fix tests

* small fixes

* fix test

* add suggestion message to comment

* add logic to upsert review comments

* update tests

* change notice to warning on check annotations

* fix tests
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.

2 participants