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

major fixes for preparation of upcoming v3.6.0 release #208

Merged
merged 41 commits into from
Mar 15, 2024
Merged

Conversation

rexut
Copy link
Member

@rexut rexut commented Mar 14, 2024

rexut added 30 commits March 7, 2024 22:42
- build: pylint to 3.1, or later
- build: regex to 2023.12, or later
- extras: pykitinfo to 1.0, or later
- documentation: breathe to 4.35, or later
- documentation: docutils to 0.18, or later
- documentation: sphinx-tsn-theme to ~2024.1

Signed-off-by: Stephan Linz <[email protected]>
This reverts commit 82879b1
since the Bridle check_compliance.py is ready to handle our
modules by MODULE_EXT_ROOT setting as well as Zephyr modules.

Signed-off-by: Stephan Linz <[email protected]>
- Checks various nits in added/modified files, not coverd by other
  checks, e.g. checkpatch.pl and pylint.
- Checks if Emails of author and signed-off messages are consistent.
- Checks for conflict markers or whitespace errors.
- Run Git, Python and YAML linting on the commits and find issues
  with style and syntax.
- Check that MAINTAINERS file parses correctly.
- Check that all modules have a MAINTAINERS entry.
- Checks introducing any unwanted properties in Devicetree Bindings.
- Checks introducing any new warnings/errors with Kconfig when no
  modules are available.
- Checks introducing any new warnings/errors with Kconfig, the basic
  Kconfig test, which is checking only for undefined references.
- Check for blocks of code or config that should be kept sorted.
- Check that any added image is limited in size.
- Check that the diff contains no binary files.

Signed-off-by: Stephan Linz <[email protected]>
The cryptographic functions defined in "u_port_crypto.h" were only required
by the now removed [in ubxlib commit 26392366cd9723826b915506e28c5a8c046d63ab]
cellular chip-to-chip security feature and hence the implementations of those
functions can now be safely deleted.  The "u_port_crypto.h" header file remains
in order not to cause backwards-compatibility issues with customer ports that
may have previously implemented the functions (and potentially used the
functions outside of ubxlib) and in case we ever require those cryptographic
functions again in the future.

Signed-off-by: Stephan Linz <[email protected]>
rexut added 11 commits March 14, 2024 15:17
Simplify the logic of version extraction by using regexes.

Signed-off-by: Stephan Linz <[email protected]>
Simplify the logic of version extraction by using regexes.

Signed-off-by: Stephan Linz <[email protected]>
Also extend the theme color set with cyan.

Signed-off-by: Stephan Linz <[email protected]>
Update all output messages throught documentation.

Signed-off-by: Stephan Linz <[email protected]>
Add 3.5.1 tag to release list to be shown on documentation side bar.

Signed-off-by: Stephan Linz <[email protected]>
Remove "Working Draft" from title.

Signed-off-by: Stephan Linz <[email protected]>
Copy link

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

@rexut rexut requested review from tobiaskaestner, Irockasingranite, andreas-kurz, a team and volker-kempert and removed request for a team and volker-kempert March 15, 2024 00:20
Copy link
Contributor

@tobiaskaestner tobiaskaestner left a comment

Choose a reason for hiding this comment

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

LGTM, many thanks

@rexut rexut merged commit abe18f5 into v3.6-branch Mar 15, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants