Skip to content

STM32 : Remove html release notes files #8844

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

Merged
merged 2 commits into from
Dec 1, 2018
Merged

Conversation

jeromecoutant
Copy link
Collaborator

Description

html releases notes files were not used at all.

Pull request type

[ ] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[x] Docs update
[ ] Test update
[ ] Breaking change

@0xc0170 0xc0170 requested a review from a team November 22, 2018 16:04
@LMESTM
Copy link
Contributor

LMESTM commented Nov 22, 2018

@jeromecoutant this was an easy way to check the HAL/LL revision - what will be the way to check the version once the change is applied ?

@bcostm
Copy link
Contributor

bcostm commented Nov 23, 2018

Agree with @LMESTM this was the purpose to add these files.

@adustm
Copy link
Member

adustm commented Nov 23, 2018

Hello, I agree with the removal of these files.
@bcostm @LMESTM You'll always be able to find STM hal version (either internally, or with github commits comments)

@bcostm
Copy link
Contributor

bcostm commented Nov 26, 2018

If these html files are removed, maybe it's worth adding a detailed description of all Cube versions in mbed inside a ST WIKI page on mbed.org.

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 26, 2018

If these html files are removed, maybe it's worth adding a detailed description of all Cube versions in mbed inside a ST WIKI page on mbed.org.

Or might be worth storing in readme in this repo (under STM32 target folder where are comon files) ? What works better for you

@bcostm
Copy link
Contributor

bcostm commented Nov 26, 2018

OK for me to create a readme file in the STM folder containing this content for example:

Summary of STM32Cube versions available in mbed:
+-------------+--------------+------------+--------------+
| STM32 Serie | Cube version | HAL driver | CMSIS driver |
+-------------+--------------+------------+--------------+
| F0          |    1.9.0     |    1.7.0   |    2.3.3     |
+-------------+--------------+------------+--------------+
| F1          |    1.6.1     |    1.1.2   |    4.2.0     |
+-------------+--------------+------------+--------------+
| F2          |    1.6.0     |    1.2.1   |    2.2.0     |
+-------------+--------------+------------+--------------+
| F3          |    1.9.0     |    1.5.0   |    2.3.2     |
+-------------+--------------+------------+--------------+
| F4          |    1.19.0    |    1.7.3   |    2.6.2     |
+-------------+--------------+------------+--------------+
| F7          |    1.10.0    |    1.2.5   |    1.2.2     |
+-------------+--------------+------------+--------------+
| L0          |    1.10.0    |    1.8.2   |    1.7.2     |
+-------------+--------------+------------+--------------+
| L1          |    1.8.1     |    1.3.1   |    2.2.3     |
+-------------+--------------+------------+--------------+
| L4          |    1.11.0    |    1.8.2   |    1.4.2     |
+-------------+--------------+------------+--------------+

Copy link
Contributor

@bcostm bcostm left a comment

Choose a reason for hiding this comment

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

Looks great. Good idea for the links.


https://www.st.com/en/embedded-software/stm32cube-mcu-packages.html

List table summarizes the STM32Cube versions available in mbed:
Copy link
Contributor

Choose a reason for hiding this comment

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

This table

@cmonr
Copy link
Contributor

cmonr commented Dec 1, 2018

CI started

@cmonr
Copy link
Contributor

cmonr commented Dec 1, 2018

Oh! Docs-only change, nice.

@cmonr cmonr merged commit 3cc7530 into ARMmbed:master Dec 1, 2018
@cmonr cmonr removed the needs: CI label Dec 1, 2018
@jeromecoutant jeromecoutant deleted the PR_HTML branch December 3, 2018 08:26
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.

6 participants