Skip to content

doc: clean up roadmap#20126

Merged
maribu merged 1 commit into
RIOT-OS:masterfrom
maribu:roadmap/cleanup
Dec 1, 2023
Merged

doc: clean up roadmap#20126
maribu merged 1 commit into
RIOT-OS:masterfrom
maribu:roadmap/cleanup

Conversation

@maribu

@maribu maribu commented Nov 30, 2023

Copy link
Copy Markdown
Member

Contribution description

  • drop completed tasks
  • drop tasks that make no sense anymore (e.g. cleanup up MIPS, as MIPS was dropped or improving xtimer, as it was superseded by ztimer)

Testing procedure

Read the markdown and the generated HTML (see link posted by Murdock).

Issues/PRs references

None

@maribu maribu added Area: doc Area: Documentation Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Nov 30, 2023
@riot-ci

riot-ci commented Nov 30, 2023

Copy link
Copy Markdown

Murdock results

✔️ PASSED

e7e50ca doc: clean up roadmap

Success Failures Total Runtime
1 0 1 59s

Artifacts

1. ~~automated unit tests with hardware in the loop (SAMR21 plugged on CI server?)~~ (done, Murdock and Philipp)
2. automated network functionality tests (e.g. RPL + UDP/PING tests through border router, multi-hop) in IoTLAB dev sites?
3. ~~leverage PiFleet more?~~ done, 1. uses PiFleet
4. On-board CI testing in IoT-LAB (as it will provide soon the possibility to add custom nodes)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@MrKevinWeiss and @miri: Could you take a look at what is done already here? Don't the automated release test cover this as well?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

If you want my feedback earlier, you need to mention @miri64 ;-). @miri got some very angry replies in the past.

And yes, the release test should cover this.

@maribu maribu Dec 1, 2023

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@miri got some very angry replies in the past.

Oopsie 😊

And yes, the release test should cover this.

Thx :-)

Comment thread doc/doxygen/src/roadmap.md Outdated
Comment thread doc/doxygen/src/roadmap.md Outdated
Comment thread doc/doxygen/src/roadmap.md
Comment thread doc/doxygen/src/roadmap.md
Comment thread doc/doxygen/src/roadmap.md Outdated
@miri64

miri64 commented Dec 1, 2023

Copy link
Copy Markdown
Member

Should we already migrate to the “vision” approach discussed during the VMA here?

miri64
miri64 previously requested changes Dec 1, 2023
Comment thread doc/doxygen/src/roadmap.md Outdated
Comment thread doc/doxygen/src/roadmap.md Outdated
- drop completed tasks
- drop tasks that make no sense anymore (e.g. cleanup up MIPS, as
  MIPS was dropped or improving xtimer, as it was superseded by
  ztimer)
@miri64 miri64 dismissed their stale review December 1, 2023 11:45

Addressed

@emmanuelsearch

emmanuelsearch commented Dec 1, 2023

Copy link
Copy Markdown
Member

I also like the idea of adding a "Rust Support" area (but same comment as above, maybe this type of restructuring should go in a separate PR).

@maribu

maribu commented Dec 1, 2023

Copy link
Copy Markdown
Member Author

I also like the idea of adding a "Rust Support" area

I also, but let's have a separate PR for this :-) Maybe @chrysn and @kaspar030 could team up for this?

@maribu maribu reopened this Dec 1, 2023
@emmanuelsearch

Copy link
Copy Markdown
Member

oops wrong button
sorry!

@maribu

maribu commented Dec 1, 2023

Copy link
Copy Markdown
Member Author

oops wrong button

I can relate. That happens to me a lot when I use the smartphone. Somehow the webpage layout keeps jumping and the button location changing while the website is not yet fully loaded. :-/

@chrysn chrysn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks to me like all questions have been addressed -- and LGTM. As extensions are for later PRs, let's get this in.

@maribu maribu added this pull request to the merge queue Dec 1, 2023
Merged via the queue into RIOT-OS:master with commit 550fef5 Dec 1, 2023
@maribu maribu deleted the roadmap/cleanup branch December 1, 2023 18:39
@maribu

maribu commented Dec 1, 2023

Copy link
Copy Markdown
Member Author

Thx :-)

@MrKevinWeiss MrKevinWeiss added this to the Release 2024.01 milestone Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: doc Area: Documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants