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

chore: prepare for 0.15.0 release #581

Merged
merged 2 commits into from Oct 24, 2022

Conversation

mdelapenya
Copy link
Collaborator

What does this PR do?

It bumps the version of the library in the docs site, and adds named parameters in a function of the public API.

Why is it important?

Keep docs up-to-date, and increase the readability of that function while consuming it.

@mdelapenya mdelapenya requested a review from a team as a code owner October 24, 2022 09:25
@mdelapenya mdelapenya self-assigned this Oct 24, 2022
@codecov
Copy link

codecov bot commented Oct 24, 2022

Codecov Report

Merging #581 (2b1f81a) into main (6b8d699) will increase coverage by 17.96%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #581       +/-   ##
===========================================
+ Coverage   15.40%   33.37%   +17.96%     
===========================================
  Files          13       13               
  Lines        1759     1759               
===========================================
+ Hits          271      587      +316     
+ Misses       1442     1087      -355     
- Partials       46       85       +39     
Impacted Files Coverage Δ
container.go 46.98% <0.00%> (+6.02%) ⬆️
docker.go 39.38% <0.00%> (+19.04%) ⬆️
docker_mounts.go 46.34% <0.00%> (+36.58%) ⬆️
logger.go 87.50% <0.00%> (+37.50%) ⬆️
mounts.go 50.00% <0.00%> (+50.00%) ⬆️
reaper.go 78.07% <0.00%> (+74.56%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@@ -60,4 +60,4 @@ nav:
- Getting help: getting_help.md
edit_uri: edit/main/docs/
extra:
latest_version: 0.14.0
latest_version: 0.15.0
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I've double checked that this value is not affecting the web UI

Screenshot 2022-10-24 at 11 34 42

@@ -12,7 +12,7 @@ import (
const defaultForSqlQuery = "SELECT 1"

//ForSQL constructs a new waitForSql strategy for the given driver
func ForSQL(port nat.Port, driver string, url func(string, nat.Port) string) *waitForSql {
func ForSQL(port nat.Port, driver string, url func(host string, port nat.Port) string) *waitForSql {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'm adding this cosmetic change here, but I can send it in a separate PR if needed

@mdelapenya
Copy link
Collaborator Author

Merging as the change is harmless

@mdelapenya mdelapenya merged commit 4c04bdf into testcontainers:main Oct 24, 2022
@mdelapenya mdelapenya deleted the prepare-0.15 branch October 24, 2022 09:54
@mdelapenya mdelapenya added the chore Changes that do not impact the existing functionality label Oct 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Changes that do not impact the existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant