Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Added F410T(8-B)Y board#1791

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
fpistm merged 6 commits intostm32duino:mainfromalextrical:STM32F4xx
Aug 30, 2022
Merged

Added F410T(8-B)Y board#1791

fpistm merged 6 commits intostm32duino:mainfromalextrical:STM32F4xx
Aug 30, 2022

Conversation

alextrical
Copy link
Contributor

Added F410T(8-B)Y board

Testing how to use Branches for development work

Copy link
Contributor

@ABOSTMABOSTM left a comment

Choose a reason for hiding this comment

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

Just to be sure we are aligned, when I said 1 PR per series, I mean 1 for all STM43F4 MCU.
But I understand that you are experimenting.
Otherwise, this PR LGTM :-)

@fpistmfpistm added the new variantAdd support of new bard labelAug 11, 2022
@fpistmfpistm added this to the2.3.1/2.4.0 milestoneAug 11, 2022
alextricaland others added2 commitsAugust 11, 2022 10:51
Co-authored-by: Frederic Pillon <frederic.pillon@st.com>
@alextrical
Copy link
ContributorAuthor

Just to be sure we are aligned, when I said 1 PR per series, I mean 1 for all STM43F4 MCU.

Correct, I just started using the smallest group I had ;)

Copy link
Contributor

@ABOSTMABOSTM left a comment

Choose a reason for hiding this comment

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

LGTM

@fpistm
Copy link
Member

Hi@alextrical
Thanks for the update anyway, a merge is not a rebase and provides artifact which should not appears in a PR.
Due to the merge you pull also several commit already available in the main.

Copy link
Member

@fpistmfpistm left a comment

Choose a reason for hiding this comment

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

LGTM

@fpistmfpistm merged commitaea5d0e intostm32duino:mainAug 30, 2022
cparata pushed a commit to cparata/Arduino_Core_STM32 that referenced this pull requestJan 31, 2023
* Added F410T(8-B)Y board* Update README.mdCo-authored-by: Frederic Pillon <frederic.pillon@st.com>* Style corrections* Corrected ldscript.ld names* Fixed issue caused by incorrect RegEx query, removing parenthesisCo-authored-by: Frederic Pillon <frederic.pillon@st.com>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@fpistmfpistmfpistm approved these changes

@ABOSTMABOSTMABOSTM approved these changes

Assignees
No one assigned
Labels
new variantAdd support of new bard
Projects
None yet
Milestone
2.4.0
Development

Successfully merging this pull request may close these issues.

3 participants
@alextrical@fpistm@ABOSTM
[8]ページ先頭
©2009-2025 Movatter.jp