Skip to content

DO NOT MERGE: Debug CI#246

Closed
ingomueller-net wants to merge 16 commits intosubstrait-io:mainfrom
ingomueller-net:debug-ci
Closed

DO NOT MERGE: Debug CI#246
ingomueller-net wants to merge 16 commits intosubstrait-io:mainfrom
ingomueller-net:debug-ci

Conversation

@ingomueller-net
Copy link
Copy Markdown
Contributor

No description provided.

dependabot Bot and others added 10 commits June 12, 2025 00:44
Bumps [substrait-java](https://github.com/substrait-io/substrait-java) from `49b0210` to `33c83dc`.
- [Release notes](https://github.com/substrait-io/substrait-java/releases)
- [Commits](substrait-io/substrait-java@49b0210...33c83dc)

---
updated-dependencies:
- dependency-name: substrait-java
  dependency-version: 33c83dc2babb4114607333d8299eced88e4afa01
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
The only change is the addition of the `Version` field, which is
expected given substrait-io/substrait-java#406.

Signed-off-by: Ingo Müller <ingomueller@google.com>
Comment thread .github/workflows/test.yml Outdated

- name: Fetch history and tags of submodules
# This is needed for `substrait-java` or one of its submodules to determine its version.
run: git submodule foreach --recursive 'git fetch --unshallow || true'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

what happens when you add the following after this command?

cd substrait-java/substrait; git describe --tags

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The tags seem to be present correctly, see CI log.

@bvolpato
Copy link
Copy Markdown
Member

Hi! This seems stale? Should we close?

@ingomueller-net
Copy link
Copy Markdown
Contributor Author

Yeah, of course. Thanks!

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants