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

fix: do not create garbage in examples nav bar #652

Merged
merged 2 commits into from Dec 5, 2022

Conversation

mdelapenya
Copy link
Collaborator

What does this PR do?

It fixes how the nav bar for the examples is generated, not appending the examples to the "non"-empty array that the generator created to define the examples nav bar.

Why is it important?

It added empty elements to the nav bar, that the user must remove before submitting the example

Related issues

@mdelapenya mdelapenya requested a review from a team as a code owner December 5, 2022 13:40
@mdelapenya mdelapenya self-assigned this Dec 5, 2022
@mdelapenya mdelapenya added chore Changes that do not impact the existing functionality bug An issue with the library and removed chore Changes that do not impact the existing functionality labels Dec 5, 2022
@mdelapenya mdelapenya requested a review from a team December 5, 2022 13:41
@mdelapenya mdelapenya removed the bug An issue with the library label Dec 5, 2022
Copy link
Member

@eddumelendez eddumelendez left a comment

Choose a reason for hiding this comment

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

just test it and it works as expected 👍🏽

@mdelapenya mdelapenya merged commit b7cdaad into testcontainers:main Dec 5, 2022
@mdelapenya mdelapenya added the chore Changes that do not impact the existing functionality label Dec 5, 2022
@mdelapenya mdelapenya deleted the fix-generator-nav-menu branch December 5, 2022 23:18
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Dec 15, 2022
* main:
  chore: generate dependabot configs for examples (testcontainers#654)
  chore: format dependabot using go-yaml indents (testcontainers#658)
  chore(deps): bump github.com/docker/compose/v2 from 2.12.2 to 2.14.0 (testcontainers#657)
  chore(deps): bump github.com/stretchr/testify from 1.8.0 to 1.8.1 (#583)
  chore(deps): bump github.com/cenkalti/backoff/v4 from 4.1.3 to 4.2.0 (testcontainers#656)
  chore: remove quotes from dependabot updates (testcontainers#655)
  fix: do not create garbage in examples nav bar (testcontainers#652)
  chore(deps): bump github.com/containerd/containerd from 1.6.8 to 1.6.10 (#621)
  chore(deps): bump golang.org/x/sys from 0.1.0 to 0.3.0 (testcontainers#651)
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

2 participants