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.

Sign up for GitHub

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

Jump to bottom

Multiline version_locations #1509

Closed
pristupa opened this issue Jul 25, 2024 · 2 comments
Closed

Multiline version_locations #1509

pristupa opened this issue Jul 25, 2024 · 2 comments
Labels
configuration use case not quite a feature and not quite a bug, something we just didn't think of

Comments

@pristupa
Copy link
Contributor

pristupa commented Jul 25, 2024 •
edited
Loading

Describe the use case
When there are many version locations it becomes difficult to read the value of the version_locations option. A new option "newline" for version_path_separator is suggested.

Example Use

version_locations =
  /foo/versions
  /bar/versions
  /baz/versions

version_path_separator = newline

Have a nice day!
@pristupa pristupa added requires triage New issue that requires categorization use case not quite a feature and not quite a bug, something we just didn't think of labels Jul 25, 2024
pristupa added a commit to pristupa/alembic that referenced this issue Jul 25, 2024
@pristupa
Enhance version_path_separator behaviour by adding a multiline option
785bdcf 
References: sqlalchemy#1509
pristupa added a commit to pristupa/alembic that referenced this issue Jul 25, 2024
@pristupa
Fix changelog ticket number
9db3d8b 
References: sqlalchemy#1509
pristupa added a commit to pristupa/alembic that referenced this issue Jul 25, 2024
@pristupa
Enhance version_path_separator behaviour by adding a multiline option
d2dca13 
References: sqlalchemy#1509

Fix changelog ticket number

References: sqlalchemy#1509
pristupa added a commit to pristupa/alembic that referenced this issue Jul 25, 2024
@pristupa
Enhance version_path_separator behaviour by adding a multiline option
6f1e86f 
References: sqlalchemy#1509
@CaselIT CaselIT added configuration and removed requires triage New issue that requires categorization labels Jul 25, 2024
@CaselIT
Copy link
Member

CaselIT commented Jul 25, 2024

hi,

seems like a reasonable request. Will look at the pr as soon as I've time

pristupa added a commit to pristupa/alembic that referenced this issue Jul 25, 2024
@pristupa
Strip leading and trailing spaces in version locations
00a1a04 
References: sqlalchemy#1509
pristupa added a commit to pristupa/alembic that referenced this issue Jul 26, 2024
@pristupa
Reduce line length to less than 80
a4d2a1e 
References: sqlalchemy#1509
pristupa added a commit to pristupa/alembic that referenced this issue Jul 27, 2024
@pristupa
Fix formatting
b1465ec 
References: sqlalchemy#1509
pristupa added a commit to pristupa/alembic that referenced this issue Jul 29, 2024
@pristupa
Update individual changelog
6155da7 
References: sqlalchemy#1509
@CaselIT
Copy link
Member

CaselIT commented Aug 5, 2024

Closed by #1510

@CaselIT CaselIT closed this as completed Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
configuration use case not quite a feature and not quite a bug, something we just didn't think of
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pristupa @CaselIT