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: update modules path in generator #877

Merged
merged 4 commits into from
Feb 27, 2023

Conversation

mdelapenya
Copy link
Member

  • fix: update tests for gh workflow modules
  • fix: update tests for docs and modules
  • fix: support passing the parent dir
  • fix: update tools.go

What does this PR do?

This PR fixes the generated code for modules in the following places:

  • the GH workflow, where the path was hardcoded to use examples.
  • the website docs for the module, where the path was also hardcoded to use examples.
  • the tools.go file, where the commet used example type

Why is it important?

The generated code, although valid in terms of the compiler, generates metadata that is incorrect. E.g. it's not possible to render the website because certain paths don't exist. At the same time, the GH workflow would fail because the path is also incorrect.

Related issues

@mdelapenya mdelapenya requested a review from a team as a code owner February 27, 2023 08:48
@mdelapenya mdelapenya added the bug An issue with the library label Feb 27, 2023
@mdelapenya mdelapenya self-assigned this Feb 27, 2023
@netlify
Copy link

netlify bot commented Feb 27, 2023

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit 07a1c49
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/63fc6e61ebf0be000898c772
😎 Deploy Preview https://deploy-preview-877--testcontainers-go.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@sonarcloud
Copy link

sonarcloud bot commented Feb 27, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

@mdelapenya mdelapenya merged commit 98eb9cd into testcontainers:main Feb 27, 2023
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Feb 27, 2023
* main:
  docs: add social links in the website (testcontainers#890)
  Update docs link for tc-node (testcontainers#878)
  fix: update modules path in generator (testcontainers#877)
@mdelapenya mdelapenya deleted the fix-generator-gh-workflow branch February 27, 2023 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An issue with the library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant