Skip to content

Repo sync #26208

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jun 23, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,6 @@ jobs:
{ name: 'graphql', path: 'src/graphql/tests', },
{ name: 'landings', path: 'src/landings/tests', },
{ name: 'linting', path: 'src/content-linter/tests', },
{ name: 'meta', path: 'tests/meta', },
{ name: 'observability', path: 'src/observability/tests' },
{ name: 'pageinfo', path: 'src/pageinfo/tests', },
{ name: 'rendering', path: 'tests/rendering', },
Expand Down
2 changes: 1 addition & 1 deletion contributing/code-annotations.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ For more experienced users, code annotations can help them understand a code exa

An annotated code example must start with a single line annotation. You can start with a blank annotation if you do not want to add an annotation to the first line of code.

You must use the comment tag for the language that the code sample is written in. For example, `#` for YAML and `//` for Ruby.
You must use the comment tag for the language that the code sample is written in. For example, `#` for YAML and `//` for JavaScript.

To enable code sample annotations, you must specify a language followed by the word `annotate` after the starting triple backtick code tag.

Expand Down
4 changes: 2 additions & 2 deletions lib/enterprise-server-releases.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ export const dates = JSON.parse(await fs.readFile('src/ghes-releases/lib/enterpr
export const next = '3.10'
export const nextNext = '3.11'

export const supported = ['3.9', '3.8', '3.7', '3.6', '3.5', '3.4']
export const supported = ['3.9', '3.8', '3.7', '3.6', '3.5']

// Ensure that:
// "next" is ahead of "latest" by one minor or major release.
Expand All @@ -37,7 +37,7 @@ function isValidNext(v1, v2) {
// When you're archiving a version, add the new archived number to this
// array and you should never need to touch the `deprecated` array
// on the line just below.
export const deprecatedWithFunctionalRedirects = ['3.3', '3.2', '3.1', '3.0']
export const deprecatedWithFunctionalRedirects = ['3.4', '3.3', '3.2', '3.1', '3.0']
export const deprecated = [
...deprecatedWithFunctionalRedirects,
'2.22',
Expand Down
1 change: 1 addition & 0 deletions src/content-linter/tests/lint-versioning.js
Original file line number Diff line number Diff line change
Expand Up @@ -157,6 +157,7 @@ function validateIfversionConditionals(conds) {
// allows us to deprecate the version before removing
// the old liquid content.
if (
version !== 'ghae' &&
!(
supported.includes(release) ||
release === next ||
Expand Down
7 changes: 6 additions & 1 deletion src/rest/scripts/test-open-api-schema.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ import frontmatter from '../../../lib/read-frontmatter.js'
import getApplicableVersions from '../../../lib/get-applicable-versions.js'
import { allVersions, getDocsVersion } from '../../../lib/all-versions.js'
import { REST_DATA_DIR, REST_SCHEMA_FILENAME } from '../lib/index.js'
import { deprecated } from '../../../lib/enterprise-server-releases.js'

const contentFiles = []

Expand Down Expand Up @@ -52,7 +53,11 @@ export async function getDiffOpenAPIContentRest() {

async function createOpenAPISchemasCheck() {
const openAPICheck = createCheckObj()
const restDirectory = fs.readdirSync(REST_DATA_DIR).filter((dir) => !dir.endsWith('.json'))
const restDirectory = fs
.readdirSync(REST_DATA_DIR)
.filter((dir) => !dir.endsWith('.json'))
// Allow the most recent deprecation to exist on disk until fully deprecated
.filter((dir) => !dir.includes(deprecated[0]))

restDirectory.forEach((dir) => {
const filename = path.join(REST_DATA_DIR, dir, REST_SCHEMA_FILENAME)
Expand Down
2 changes: 1 addition & 1 deletion tests/fixtures/content/get-started/liquid/ifversion.md
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ condition-l

{% ifversion ghes %}
condition-m
{% ifversion ghes = 3.4 %}
{% ifversion ghes = 3.5 %}
condition-n
{% endif %}
{% endif %}
Expand Down
21 changes: 0 additions & 21 deletions tests/meta/orphan-tests.js

This file was deleted.

10 changes: 1 addition & 9 deletions tests/rendering-fixtures/liquid.js
Original file line number Diff line number Diff line change
Expand Up @@ -154,15 +154,6 @@ describe('extended Markdown', () => {
'condition-l',
],
'enterprise-cloud@latest': ['condition-c', 'condition-j', 'condition-l'],
'[email protected]': [
'condition-c',
'condition-e',
'condition-g',
'condition-i',
'condition-j',
'condition-m',
'condition-n',
],
'[email protected]': [
'condition-c',
'condition-e',
Expand All @@ -172,6 +163,7 @@ describe('extended Markdown', () => {
'condition-i',
'condition-k',
'condition-m',
'condition-n',
'condition-o',
],
'[email protected]': [
Expand Down