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

Adding comment surrounding usage of Oracle TNS Connection string #1854

Merged
merged 2 commits into from Dec 4, 2017

Conversation

@cdavid15
Copy link
Contributor

@cdavid15 cdavid15 commented Dec 2, 2017

Minor update to the example flyway.conf file bundled with the command line to explicitly show connections to oracle databases can be made using TNS names

@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Dec 3, 2017

Good idea. It makes sense to make this more explicit.

@axelfontaine axelfontaine added this to the Flyway 5.0.0 milestone Dec 3, 2017
@axelfontaine axelfontaine merged commit 6e54baa into flyway:master Dec 4, 2017
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
license/cla Contributor License Agreement is signed.
Details
axelfontaine added a commit that referenced this pull request Dec 4, 2017
axelfontaine added a commit to flyway/flywaydb.org that referenced this pull request Dec 4, 2017
axelfontaine added a commit to flyway/flywaydb.org that referenced this pull request Dec 4, 2017
selliera pushed a commit to selliera/flyway that referenced this pull request Dec 7, 2017
Adding comment surrounding usage of Oracle TNS Connection string
selliera pushed a commit to selliera/flyway that referenced this pull request Dec 7, 2017
dohrayme pushed a commit to dohrayme/flyway that referenced this pull request Feb 3, 2020
Adding comment surrounding usage of Oracle TNS Connection string
dohrayme pushed a commit to dohrayme/flyway that referenced this pull request Feb 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants