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

Prevent empty lines in schema.rb if db has no enums #60

Conversation

Mayurifag
Copy link
Contributor

Context

Gem itself is done in context that application has at least one enum.

If someone drops all postgres enum related things via migration and dumps the schema.rb, he will have 2 weird empty lines, which he would not have, if he had no gem.

So I've done this little request, so no empty lines would be added in schema.rb if there are no enums.

@bibendi
Copy link
Owner

bibendi commented Jan 12, 2024

Hi! Thank you for the contribution!

@bibendi
Copy link
Owner

bibendi commented Jan 12, 2024

It seems, we need to drop support for Ruby 2.6

@Mayurifag Mayurifag force-pushed the prevent-empty-lines-in-schema-rb-without-enums branch from 9b17bbe to 2a8f5c8 Compare January 12, 2024 07:32
@Mayurifag
Copy link
Contributor Author

Mayurifag commented Jan 12, 2024

I've forcepushed my last commit, — added drop 2.6 ruby due to dip requirement of 2.7 version

I've also removed buster part in docker-compose.yml — there is no ruby:3.3-buster docker image.

.standard.yml Show resolved Hide resolved
@Mayurifag Mayurifag force-pushed the prevent-empty-lines-in-schema-rb-without-enums branch from d799c98 to ec2d422 Compare January 12, 2024 11:08
@Mayurifag Mayurifag force-pushed the prevent-empty-lines-in-schema-rb-without-enums branch from ec2d422 to 2e0771f Compare January 17, 2024 17:58
@bibendi bibendi merged commit d2ab897 into bibendi:master Jan 18, 2024
5 checks passed
@Mayurifag Mayurifag deleted the prevent-empty-lines-in-schema-rb-without-enums branch January 23, 2024 14:20
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.

None yet

2 participants