Skip to content

Flag experimental - #1023

Merged
FloThinksPi merged 3 commits into
mainfrom
flag-experimental
Aug 8, 2025
Merged

Flag experimental#1023
FloThinksPi merged 3 commits into
mainfrom
flag-experimental

Conversation

@FloThinksPi

Copy link
Copy Markdown
Member

No description provided.

@FloThinksPi
FloThinksPi force-pushed the flag-experimental branch 9 times, most recently from aef1fe8 to 6fb0d2c Compare August 8, 2025 14:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds an experimental flag to the repository and introduces a GitHub issue management script. The main purpose is to flag the project as experimental and provide tooling for automated OpenAPI specification quality checking.

  • Adds experimental notice to README with disclaimer about production use
  • Introduces automated issue management script for OpenAPI quality checks
  • Updates file references to use consistent PascalCase naming convention
  • Removes dredd dependency from package.json and updates GitHub Actions to use Alpine containers

Reviewed Changes

Copilot reviewed 52 out of 72 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
README.md Added experimental flag and disclaimer, updated command examples
scripts/manage_issues.js New automated script for creating GitHub issues to track OpenAPI quality checks
scripts/enpoints.txt List of API endpoints for the quality checking script
package.json Removed dredd dependency
.github/workflows/*.yaml Updated to use Alpine containers and improved lint configuration
apis/cf/latest/**/*.yaml Standardized file references to use PascalCase naming

Comment thread scripts/manage_issues.js
const RATE_LIMIT_DELAY = 10000; // 10 seconds between API calls
const MAX_RETRIES = 10;

const endpoints = fs.readFileSync(path.join(__dirname, 'enpoints.txt'), 'utf-8');

Copilot AI Aug 8, 2025

Copy link

Choose a reason for hiding this comment

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

The filename 'enpoints.txt' is misspelled. It should be 'endpoints.txt'.

Suggested change
const endpoints = fs.readFileSync(path.join(__dirname, 'enpoints.txt'), 'utf-8');
const endpoints = fs.readFileSync(path.join(__dirname, 'endpoints.txt'), 'utf-8');

Copilot uses AI. Check for mistakes.
Comment thread scripts/manage_issues.js
- [ ] **Clarity and Accuracy**: Check for clarity, accuracy, and completeness in the summary and description.
- [ ] **Concise Summary**: Ensure the \`summary\` provides a short, easy-to-understand overview of the operation.
- [ ] **Detailed Description**: Verify the \`description\` provides enough detail, including any specific behaviors or constraints.
- [ ] **GithubMarkdown Syntax**: Ensure that GithubMarkdown syntax is used correctly for rich text representation.`

Copilot AI Aug 8, 2025

Copy link

Choose a reason for hiding this comment

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

The term 'GithubMarkdown' should be 'GitHub Markdown' (with proper capitalization and spacing).

Suggested change
- [ ] **GithubMarkdown Syntax**: Ensure that GithubMarkdown syntax is used correctly for rich text representation.`
- [ ] **GitHub Markdown Syntax**: Ensure that GitHub Markdown syntax is used correctly for rich text representation.`

Copilot uses AI. Check for mistakes.
@jochenehret
jochenehret self-requested a review August 8, 2025 15:03
@FloThinksPi
FloThinksPi merged commit e565c79 into main Aug 8, 2025
2 checks passed
@FloThinksPi
FloThinksPi deleted the flag-experimental branch August 8, 2025 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants