Skip to content
This repository was archived by the owner on Dec 18, 2024. It is now read-only.

Databroker Git version information in container seems to be incorrect #516

Closed
SebastianSchildt opened this issue Mar 20, 2023 · 3 comments
Closed
Labels
bug Something isn't working might not get fixed These issues might not get fixed and are just for documentation purposes here

Comments

@SebastianSchildt
Copy link
Contributor

This came up in review of #506

Solution are not elegant, but easy

In git checkout action no tags are available, and can neither be checked, options are

@lukasmittag lukasmittag added bug Something isn't working might not get fixed These issues might not get fixed and are just for documentation purposes here labels Apr 18, 2023
@SebastianSchildt
Copy link
Contributor Author

Maybe we should think about it in terms of Github Releases: Those should be automated via GH actions, and there we shoudl decide the "correct" way to determine version information

@erikbosch
Copy link
Contributor

As a comment. For kuksa-python-sdk we do not do a shallow clone in CI. Is the size actually a problem?

    - uses: actions/checkout@v4
      with:
        # Fetch everything to get tags working as expected
        fetch-depth: 0
        submodules: 'true'

@erikbosch
Copy link
Contributor

We are about to archive this repo soon. If you consider this issue as important please file a new issue at one of the new Kuksa repos at https://github.com/eclipse-kuksa

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
bug Something isn't working might not get fixed These issues might not get fixed and are just for documentation purposes here
Projects
None yet
Development

No branches or pull requests

3 participants