Skip to content
This repository was archived by the owner on May 14, 2026. It is now read-only.
This repository was archived by the owner on May 14, 2026. It is now read-only.

showcase samples should be part of golden tests #2470

Closed

Description

@zhumin8

sample snippets here should be part of the golden testing for Showcase.

Steps to identify this:

rm -rf showcase/gapic-showcase/samples/
mvn clean install -DskipTests
cd showcase

then mvn compile -P update, expect the samples should be re-generated and added back.
But this samples folder is not re-generated

Activity

  1. added
    type: feature request‘Nice-to-have’ improvement, new feature or different behavior or design.
    priority: p3Desirable enhancement or fix. May not be included in next release.
    on Feb 15, 2024
  2. added
    priority: p2Moderately-important priority. Fix may not be included in next release.
    type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.
    and removed
    type: feature request‘Nice-to-have’ improvement, new feature or different behavior or design.
    on Mar 4, 2024
  3. blakeli0 commented on Mar 4, 2024

    @blakeli0
    Contributor

    I think it worked before, could it be an issue caused by migrating Showcase to the hermetic build scrips? cc: @diegomarquezp @JoeWang1127

  4. JoeWang1127 commented on Mar 4, 2024

    @JoeWang1127
    Contributor
  5. removed
    priority: p3Desirable enhancement or fix. May not be included in next release.
    on Mar 4, 2024
  6. lqiu96 commented on May 21, 2024

    @lqiu96
    Member

    Keeping this a P2 issue. Seems to be due to a regression in the hermetic build scripts.

  7. blakeli0 commented on Aug 6, 2024

    @blakeli0
    Contributor

    This was done in the early phase of hermetic build when we didn't have the image. We should migrate the showcase module to use hermetic build image now, with a dedicated generation_config.yaml.

  8. self-assigned this
    on Aug 23, 2024
  9. diegomarquezp commented on Jan 16, 2025

    @diegomarquezp
    Contributor

    #3568 uses the docker image and generates samples

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

priority: p2Moderately-important priority. Fix may not be included in next release.samplesIssues that are directly related to samples.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions