Backfill GraphQL epic type descriptions (part 2)
What does this MR do?
Continuation of !19233 (merged)
Backfill some more GraphQL fields, this time for the Epic type.
Description standards:
-
{x} of the {y}
where possible !19233 (comment 236991402) - Booleans framed in terms of "what does this field do" i.e.
Indicates repository has a Git repository
!19233 (comment 236991402) - Always use
timestamp
when describing aTypes::TimeType
field !19233 (comment 236997153) - No
.
at end of strings
Related to #31645 (closed)
Merge request reports
Activity
changed milestone to %12.6
added GraphQL backstage [DEPRECATED] documentation labels
mentioned in issue #31645 (closed)
Reviewer roulette
Changes that require review have been detected! A merge request is normally reviewed by both a reviewer and a maintainer in its primary category (e.g. frontend or backend), and by a maintainer in all other categories.
To spread load more evenly across eligible reviewers, Danger has randomly picked a candidate for each review slot. Feel free to override this selection if you think someone else would be better-suited, or the chosen person is unavailable.
Once you've decided who will review this merge request, mention them as you normally would! Danger does not (yet?) automatically notify them for you.
Category Reviewer Maintainer backend Dylan Griffith ( @DylanGriffith
)Stan Hu ( @stanhu
)Generated by
DangerEdited by 🤖 GitLab Bot 🤖@felipe_artur would you please review?
assigned to @felipe_artur
- Resolved by charlie ablett
- Resolved by charlie ablett
- Resolved by charlie ablett
- Resolved by Felipe Cardozo
@cablett Thanks a lot for opening this!!!! LGTM!
I left just one small suggestion.
unassigned @felipe_artur
assigned to @jprovaznik
- Resolved by charlie ablett
@jprovaznik would you please perform maintainer review?
@eread would you please perform docs review?
Thank you both
- Resolved by charlie ablett
- Resolved by charlie ablett
unassigned @jprovaznik
- Resolved by charlie ablett
assigned to @eread
assigned to @jprovaznik and unassigned @eread and @cablett
added backend label
Thanks @cablett, LGTM
Could you please rebase this MR (we should not merge MR's which are >100 commits behind master if master is broken)? I tried to rebase locally myself to speed things up, but hit some conflicts.
assigned to @cablett and unassigned @jprovaznik
added 442 commits
-
fb1b8008...0123a8ea - 441 commits from branch
master
- 09d03b0a - Backfill epic type descriptions
-
fb1b8008...0123a8ea - 441 commits from branch
Thanks @cablett, setting MWPS.
enabled an automatic merge when the pipeline for 09d03b0a succeeds
assigned to @jprovaznik
mentioned in commit cd3d55f7
mentioned in merge request !21536 (merged)
mentioned in merge request !21809 (merged)
mentioned in merge request !22073 (merged)
mentioned in merge request !22199 (merged)
mentioned in merge request !24716 (merged)
added docsimprovement label and removed 1 deleted label
added typemaintenance label