Skip to content

Conversation

@Madhunika-atmakuri
Copy link
Contributor

@Madhunika-atmakuri Madhunika-atmakuri commented Nov 11, 2025

Description

@swift-nav/algint-team
This PR fixes the release failures:

  1. Modified the HOWTO script to use the new libsbp build image
  2. Adds sbp.jit file to exclude patterns since it was removed in a recent previous PR but Sphinx throws an error about it

API compatibility

No, just minor fixes

JIRA Reference

https://swift-nav.atlassian.net/browse/BOARD-XXXX

@Madhunika-atmakuri Madhunika-atmakuri force-pushed the madhunika/fix_release_failures branch from e89d44c to 4cbea89 Compare November 11, 2025 21:00
@Madhunika-atmakuri Madhunika-atmakuri marked this pull request as ready for review November 11, 2025 21:00
@Madhunika-atmakuri Madhunika-atmakuri requested a review from a team as a code owner November 11, 2025 21:00
# List of patterns, relative to source directory, that match files and
# directories to ignore when looking for source files.
exclude_patterns = []
exclude_patterns = ['sbp.jit.rst']
Copy link
Contributor Author

Choose a reason for hiding this comment

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

sbp.jit was removed in #1519 by @dgburr
but Sphinx auto-discovers it and throws an error since it's not in the toctree. Adding this to exclude patterns fixed the release script. I'm not sure if there any permanent fix for this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Apologies. It turns out to be caching issue. I will revert this change

@sonarqubecloud
Copy link

@Madhunika-atmakuri Madhunika-atmakuri marked this pull request as draft November 11, 2025 22:44
Copy link
Contributor

@dgburr dgburr left a comment

Choose a reason for hiding this comment

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

Yeah I got caught out by those toc caching changes as well.

But good catch on corrected the HOWTO.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants