Skip to content

doc/rdm: clarification to revision process#10798

Merged
jia200x merged 1 commit intoRIOT-OS:masterfrom
danpetry:doc/rdm0_process_clarification
Jan 17, 2019
Merged

doc/rdm: clarification to revision process#10798
jia200x merged 1 commit intoRIOT-OS:masterfrom
danpetry:doc/rdm0_process_clarification

Conversation

@danpetry
Copy link
Copy Markdown
Contributor

@danpetry danpetry commented Jan 17, 2019

Contribution description

This PR attempts to clarify the difference between a change that can be addressed by minor revision and one that must be addressed by deprecation.

Testing procedure

Read!

@kaspar030 kaspar030 added Process: needs >1 ACK Integration Process: This PR requires more than one ACK Area: RDM Area: Discussions on RIOT Developer Memos labels Jan 17, 2019
kaspar030
kaspar030 approved these changes Jan 17, 2019
View reviewed changes
Copy link
Copy Markdown
Contributor

@kaspar030 kaspar030 left a comment

Choose a reason for hiding this comment

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

ACK. Nice example of clarifiying an RDM. 😉

@danpetry danpetry mentioned this pull request Jan 17, 2019
Open
@danpetry
Copy link
Copy Markdown
Contributor Author

danpetry commented Jan 17, 2019

meta!

@waehlisch waehlisch self-requested a review January 17, 2019 11:22
waehlisch
waehlisch approved these changes Jan 17, 2019
View reviewed changes
Copy link
Copy Markdown
Member

@waehlisch waehlisch left a comment

Choose a reason for hiding this comment

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

Thanks for the clarification!

@jia200x jia200x added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Jan 17, 2019
miri64
miri64 approved these changes Jan 17, 2019
View reviewed changes
Copy link
Copy Markdown
Member

@miri64 miri64 left a comment

Choose a reason for hiding this comment

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

ACK

jia200x
jia200x approved these changes Jan 17, 2019
View reviewed changes
Copy link
Copy Markdown
Member

@jia200x jia200x left a comment

Choose a reason for hiding this comment

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

ACK

@jia200x jia200x merged commit 69d4235 into RIOT-OS:master Jan 17, 2019
@danpetry danpetry deleted the doc/rdm0_process_clarification branch January 17, 2019 14:06
@emmanuelsearch
Copy link
Copy Markdown
Member

emmanuelsearch commented Jan 17, 2019

thanks @danpetry

@aabadie aabadie added this to the Release 2019.01 milestone Jan 24, 2019
@danpetry danpetry restored the doc/rdm0_process_clarification branch April 29, 2019 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@miri64 miri64 miri64 approved these changes

@waehlisch waehlisch waehlisch approved these changes

@jia200x jia200x jia200x approved these changes

@kaspar030 kaspar030 kaspar030 approved these changes

@emmanuelsearch emmanuelsearch Awaiting requested review from emmanuelsearch

Assignees

No one assigned

Labels

Area: RDM Area: Discussions on RIOT Developer Memos CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: needs >1 ACK Integration Process: This PR requires more than one ACK

Projects

None yet

Milestone

Release 2019.01

Development

Successfully merging this pull request may close these issues.

7 participants

@danpetry @emmanuelsearch @miri64 @waehlisch @jia200x @kaspar030 @aabadie