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

Move banner.txt to new location #1496

Merged
merged 1 commit into from
Nov 3, 2020
Merged

Move banner.txt to new location #1496

merged 1 commit into from
Nov 3, 2020

Conversation

timurt
Copy link
Contributor

@timurt timurt commented Oct 19, 2020


name: Move banner.txt to new location
about: #1476
title: ''
labels: Status:Discovery
assignees: ''


Environment

Liquibase Version: 4.1.2-local-SNAPSHOT

Liquibase Integration & Version: CLI

Liquibase Extension(s) & Version: -

Database Vendor & Version: -

Operating System Type & Version: Mac OS X 10.13.6

Pull Request Type

  • Bug fix (non-breaking change which fixes an issue.)
  • Enhancement/New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Description

Followed by #1476

Could you please consider an alternative location for this resource? I am working on the Spring Boot project and we use this location for users that wants to use an alternative banner when starting their apps. Arguably, our default isn't great either and we should probably fix it.

Actual Behavior

Actual location of the banner.txt is under src/main/resources folder

Expected/Desired Behavior

New location of the banner.txt is under src/main/resources/liquibase folder

Fast Track PR Acceptance Checklist:

@timurt
Copy link
Contributor Author

timurt commented Oct 19, 2020

I moved banner.txt to liquibase subfolder, another approach was to rename banner.txt to banner-liquibase.txt

Both ways seem legitimate to me, since initial problem will be solved

@timurt
Copy link
Contributor Author

timurt commented Oct 19, 2020

And could you please add hacktoberfest-accepted to this PR

@codecov
Copy link

codecov bot commented Oct 19, 2020

Codecov Report

Merging #1496 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1496   +/-   ##
=========================================
  Coverage     47.96%   47.96%           
- Complexity     7871     7872    +1     
=========================================
  Files           793      793           
  Lines         38456    38456           
  Branches       6901     6901           
=========================================
+ Hits          18446    18447    +1     
  Misses        17565    17565           
+ Partials       2445     2444    -1     
Impacted Files Coverage Δ Complexity Δ
...base/integration/commandline/CommandLineUtils.java 18.33% <100.00%> (ø) 4.00 <0.00> (ø)
...re/src/main/java/liquibase/util/ISODateFormat.java 80.32% <0.00%> (+1.63%) 17.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bcb4926...a7024b7. Read the comment docs.

@molivasdat molivasdat linked an issue Oct 19, 2020 that may be closed by this pull request
@molivasdat molivasdat added hacktoberfest-accepted hacktoberfest a month-long celebration of open-source software and Developers contribute by completing PRs labels Oct 21, 2020
@nvoxland
Copy link
Contributor

nvoxland commented Nov 3, 2020

Thanks!
Cherry picked to 4.2.x as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest a month-long celebration of open-source software and Developers contribute by completing PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider alternative location for banner.txt
3 participants