Skip to content

Conversation

@jtapia
Copy link
Collaborator

@jtapia jtapia commented Jul 8, 2023

No description provided.

@jtapia jtapia force-pushed the support/v4 branch 2 times, most recently from 47a04e7 to 1d97ab3 Compare July 8, 2023 03:57
Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution. Could you please create single commits for each change and explain why a change was necessary. Also, there is a change in tested ruby version. Why has this changed be made and can you please revert it?

executor:
name: solidusio_extensions/postgres
ruby_version: "3.2"
ruby_version: "3.0"
Copy link
Member

Choose a reason for hiding this comment

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

Why have you changed the tested ruby version?

run-specs-with-mysql:
executor:
name: solidusio_extensions/mysql
ruby_version: "3.1"
Copy link
Member

Choose a reason for hiding this comment

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

Same here. Please do not change test config

recent activity. It will be closed if no further activity occurs. Thank you
for your contributions.
# Comment to post when closing a stale issue. Set to `false` to disable
closeComment: false No newline at end of file
Copy link
Member

Choose a reason for hiding this comment

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

Is this change related to the Solidus 4.0 upgrade? If yes, can you explain why?

@@ -0,0 +1,2 @@
issues=false
exclude-labels=infrastructure
Copy link
Member

Choose a reason for hiding this comment

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

Where does this change came from? Can you make this a separate commit.

Rails/SkipsModelValidations:
Exclude:
- db/migrate/**/*
NewCops: disable
Copy link
Member

Choose a reason for hiding this comment

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

This change also qualifies a separate commit

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.

3 participants