RTEMS / Documentation / Documentation

Go to Issues or Merge Requests


Issues

6 - Merge Request Approval process is broken

Id

6

State

closed

Type

ISSUE

Author

Gedare Bloom

Assignee(s)

Amar Takhar, Gedare Bloom

Closed by

Amar Takhar

Created

2024-05-02T03:44:41.017Z

Closed

2024-07-08T04:18:12.604Z

Updated

2024-07-08T04:18:12.616Z

Milestone

6.2

Link

https://gitlab.rtems.org/rtems/docs/rtems-docs/-/issues/6

Merges

2

Summary

The approver flow in this repo requires 1 general approver and does not require 1 maintainer approver.

See also https://gitlab.rtems.org/administration/gitlab/-/issues/6

Steps to reproduce

Author: Gedare Bloom

2024-05-02T03:44:41.109Z

assigned to @gedare

Author: Amar Takhar

2024-05-02T03:57:25.218Z

marked this issue as related to administration/gitlab#6

Author: Amar Takhar

2024-05-02T03:57:54.881Z

FYI the ‘linked items’ section is handy keeps things more organised and will combine certain things

Author: Gedare Bloom

2024-05-02T04:07:03.082Z

created branch 6-merge-request-approval-process-is-broken to address this issue

Author: Gedare Bloom

2024-05-02T04:07:34.064Z

mentioned in merge request !6

Author: Gedare Bloom

2024-05-02T04:51:03.785Z

mentioned in merge request !7

Author: Gedare Bloom

2024-05-02T05:18:58.469Z

See https://gitlab.rtems.org/gedare/rtems-docs/-/commit/e59b13dd40f52a3d831967795b70a1681acbb84b for a proof-of-concept patch.

The basic logic is to make the General Approvers section optional, and to require at least 1 approval from the Maintainer Approvers section.

Now, since only 1 approval is required in the repo settings, this means the General Approvers approval is purely optional/advisory, as at least 1 Maintainer would still need to approve an MR.

Author: Amar Takhar

2024-05-17T00:39:46.439Z

assigned to @amar

Author: Amar Takhar

2024-07-08T04:18:12.376Z

We have been running this for over a month now it is all working as we want.

93 - rtems-ver-minor set wrong in release? (opened)

Id

93

State

opened

Type

ISSUE

Author

Gedare Bloom

Assignee(s)

Gedare Bloom

Created

2025-02-27T16:51:58.067Z

Updated

2025-10-02T22:36:06.097Z

Milestone

6.2

Labels

doc, release

Link

https://gitlab.rtems.org/rtems/docs/rtems-docs/-/issues/93

Merges

0

Summary

The minor number is 0 in the released docs for RTEMS 6.1

I’m not sure if this belongs to docs or release.

See for example https://docs.rtems.org/docs/6.1/user/start/sources.html#releases

@chris

Author: Gedare Bloom

2025-02-27T16:51:58.192Z

assigned to @gedare

Author: Chris Johns

2025-02-28T05:39:05.939Z

The 6.1 released docs can be found here https://ftp.rtems.org/pub/rtems/releases/6/6.1/docs/html/user/index.html and they look OK to me:

!`image.png <https://gitlab.rtems.org/-/project/uploads/0bd474ff5290a8c38a7b1783e937d97e/image.png>`_

@amar ?

Author: Chris Johns

2025-02-28T05:54:50.923Z

Checked docs.rtems.org and it also looks OK.

Author: Chris Johns

2025-10-02T22:36:06.067Z

added rtems#10 as parent epic

73 - posix_rst.py: Correct Spelling of Application (opened)

Id

73

State

opened

Type

ISSUE

Author

Joel Sherrill

Assignee(s)

Joel Sherrill

Created

2025-01-27T22:35:09.249Z

Updated

2025-01-27T23:52:11.785Z

Milestone

6.2

Link

https://gitlab.rtems.org/rtems/docs/rtems-docs/-/issues/73

Merges

0

Summary

This issue is to correct the spelling of “Application” in the posix_rst.py program. This spelling error propagates into the headings of the generated documentation.

Pre-set options

Author: Joel Sherrill

2025-01-27T22:35:09.411Z

assigned to @joel

Author: Joel Sherrill

2025-01-27T22:35:13.341Z

Author: Joel Sherrill

2025-01-27T23:52:11.701Z

mentioned in merge request !141

Merge Requests

141 - posix-compliance/posix_rst.py: Spell application correctly

Id

141

State

merged

Merged by

Kinsey Moore

Merged at

2025-01-28T01:56:52.534Z

Assignee(s)

Joel Sherrill

Created

2025-01-27T23:52:05.883Z

Updated

2025-01-28T01:56:53.516Z

Milestone

6.2

Link

https://gitlab.rtems.org/rtems/docs/rtems-docs/-/merge_requests/141

Commit

9ad29e1c674f2da9a6a6fad9a921bd8939abd525

Summary

posix-compliance/posix_rst.py: Spell application correctly

Closes #73

Author: Joel Sherrill

2025-01-27T23:52:06.085Z

assigned to @joel

Author: Kinsey Moore

2025-01-28T01:56:45.539Z

approved this merge request

92 - README: Update to book theme

Id

92

State

merged

Merged by

Chris Johns

Merged at

2024-11-22T06:16:51.135Z

Reviewer(s)

Chris Johns

Created

2024-11-22T06:11:14.520Z

Updated

2024-11-22T06:16:52.219Z

Milestone

6.2

Link

https://gitlab.rtems.org/rtems/docs/rtems-docs/-/merge_requests/92

Commit

fffa54e29449d48d3e4cd66a309a5ea4630fc5ba

Summary

README: Update to book theme

(cherry picked from commit 3aa5a1aeef00fab266c07c4a732401f28a1eec45)

7ec1823f README: Update to book theme 3c7eb1e2 Update README.md 7619a2c7 Update README.md

Co-authored-by: Chris Johns <chrisj@rtems.org>

Author: Gedare Bloom

2024-11-22T06:11:14.682Z

requested review from @chris

Author: Chris Johns

2024-11-22T06:16:44.670Z

approved this merge request