Skip to content

feat: notify platform-types repo on reference change - #853

Merged
rvab merged 22 commits into
mainfrom
feat/notify-types-on-spec-change
Jul 28, 2026
Merged

feat: notify platform-types repo on reference change#853
rvab merged 22 commits into
mainfrom
feat/notify-types-on-spec-change

Conversation

@rvab

@rvab rvab commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Description

Please add PR description here, add screenshots if needed

Clickup

app.clickup.com

Copilot AI review requested due to automatic review settings July 28, 2026 05:47
@github-actions github-actions Bot added the size/S Small PR label Jul 28, 2026

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

Adds a GitHub Actions workflow to notify the fylein/fyle-platform-types repository whenever bundled OpenAPI specs in reference/ change on main.

Changes:

  • Introduces a new workflow that triggers on push to main affecting reference/**.
  • Generates a GitHub App installation token and sends a repository_dispatch event (specs-updated) to fyle-platform-types with commit metadata.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/notify-types-repo.yml
Copilot AI review requested due to automatic review settings July 28, 2026 05:49
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (1)

.github/workflows/notify-types-repo.yml:10

  • on.push.branches is defined as a scalar (branches: main). In this repo’s other workflows, branches is consistently a YAML sequence (e.g. .github/workflows/secrets-scan.yml:9-10 and bundler.yml:9-10). Using the sequence form here avoids any ambiguity in GitHub Actions event filtering and keeps the workflow consistent with the codebase.
on:
  push:
    branches:
      - main
    paths:

Copilot AI review requested due to automatic review settings July 28, 2026 05:51
@rvab
rvab requested a review from Aniruddha-Shriwant July 28, 2026 05:52

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings July 28, 2026 08:09

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings July 28, 2026 11:54

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (2)

.github/workflows/notify-types-repo.yml:14

  • This workflow doesn’t appear to need any repository-scoped GITHUB_TOKEN permissions because it only uses repository secrets and an installation token generated from those secrets. Granting contents: read is broader than necessary; consider setting permissions to none to follow least-privilege.
permissions:
  contents: read

.github/workflows/notify-types-repo.yml:1

  • The PR description still contains the placeholder text (“Please add PR description here…”) and the ClickUp section is only a bare domain. This repo’s contribution guidelines require a real ClickUp link in the PR description; please update the PR description accordingly.
name: Notify types repo on spec change

@rvab
rvab merged commit 762d5e1 into main Jul 28, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S Small PR

Development

Successfully merging this pull request may close these issues.

3 participants