Skip to content

Commit 46e8acf

Browse files
authored
Remove trailing mentions of GHAE in code (#49429)
1 parent 15da689 commit 46e8acf

File tree

4 files changed

+3
-13
lines changed

4 files changed

+3
-13
lines changed

src/data-directory/lib/data-schemas/features.js

-3
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,6 @@ delete featureVersions.properties.versions.properties.feature
1414
// Call it invalid if any properties other than version properties are found.
1515
featureVersions.additionalProperties = false
1616

17-
// TODO - UNCOMMENT THE FOLLOWING LINE WHEN GHAE IS UPDATED WITH SEMVER VERSIONING
18-
// featureVersions.properties.versions.additionalProperties = false
19-
2017
// avoid ajv strict warning
2118
featureVersions.type = 'object'
2219

src/release-notes/components/types.ts

-6
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,6 @@ export type ReleaseNotePatch = {
3333
sections: Record<string, Array<ReleaseNoteSection>>
3434
}
3535

36-
export type GHAEReleaseNotesContextT = {
37-
releaseNotes: Array<ReleaseNotePatch>
38-
releases: Array<{ version: string; patches: Array<ReleaseNotePatch> }>
39-
currentVersion: CurrentVersion
40-
}
41-
4236
export type GHESReleaseNotesContextT = {
4337
latestPatch: string
4438
latestRelease: string

src/release-notes/middleware/ghes-release-notes.js

-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ export default async function ghesReleaseNotesContext(req, res, next) {
88
const [requestedPlan, requestedRelease] = req.context.currentVersion.split('@')
99
if (requestedPlan !== 'enterprise-server') return next()
1010

11-
// (This applies to ghae release notes too)
1211
// We deliberately force the language to be English for now.
1312
// The underlying reason is that the content (in data/release-notes/**/*.yml)
1413
// is Markdown that does NOT use variables controlled by English-only

src/workflows/content-changes-table-comment.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ async function main(owner, repo, baseSHA, headSHA) {
107107
try {
108108
// the try/catch is needed because getApplicableVersions() returns either [] or throws an error when it can't parse the versions frontmatter
109109
// try/catch can be removed if docs-engineering#1821 is resolved
110-
// i.e. for feature based versioning, like ghae: 'issue-6337'
110+
// i.e. for feature based versioning, like ghec: 'issue-6337'
111111
const fileVersions = getApplicableVersions(data.versions)
112112

113113
for (const plan in allVersionShortnames) {
@@ -120,15 +120,15 @@ async function main(owner, repo, baseSHA, headSHA) {
120120
)
121121

122122
if (versions.length === 1) {
123-
// for fpt, ghec, and ghae
123+
// for fpt and ghec
124124

125125
if (versions.toString() === nonEnterpriseDefaultVersion) {
126126
// omit version from fpt url
127127

128128
previewCell += `[${plan}](${APP_URL}/${fileUrl})<br>`
129129
prodCell += `[${plan}](${PROD_URL}/${fileUrl})<br>`
130130
} else {
131-
// for non-versioned releases (ghae, ghec) use full url
131+
// for non-versioned releases (ghec) use full url
132132

133133
previewCell += `[${plan}](${APP_URL}/${versions}/${fileUrl})<br>`
134134
prodCell += `[${plan}](${PROD_URL}/${versions}/${fileUrl})<br>`

0 commit comments

Comments
 (0)