Skip to content
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

Feat: update spreadsheet worflow setup #30

Merged
merged 1 commit into from
Oct 11, 2024

Conversation

GarvitSinghal47
Copy link
Contributor

Description

This PR implements a workflow that triggers on specific pull request (PR) and review events. The workflow collects and updates PR data into a spreadsheet.

Issue addressed

Addresses #PR# HERE

Before/after screenshots

Changelog

  • Description: Added workflow to handle PR events (open, edit, close, synchronize, review events, and comments) and update a spreadsheet with key PR details.
  • Products impact: Internal update for managing PR metadata.
  • Addresses: -
  • Components: .github workflows.
  • Breaking: no
  • Impacts a11y: no
  • Guidance: This update adds automation for tracking PR details and should be used for all relevant repositories moving forward.

Steps to test

  1. Open a new PR or edit an existing PR.
  2. Ensure the PR metadata is updated in the linked spreadsheet.
  3. Review the workflow execution for any errors in the GitHub Actions tab.

(optional) Implementation notes

At a high level, how did you implement this?

Created a GitHub Action workflow to capture PR event data and trigger the update-pr-spreadsheet.yml workflow, which populates PR details into a spreadsheet.

Does this introduce any tech-debt items?

No, this is a self-contained update to PR tracking automation.

Testing checklist

  • Contributor has fully tested the PR manually by creating and editing PRs.
  • Workflow execution is error-free in GitHub Actions.
  • The changelog item is included in the PR.

Reviewer guidance

  • Is the workflow properly configured to trigger on relevant PR events?
  • Are the data fields captured from the PR accurate and complete?
  • Is the documentation for the workflow clear and sufficient?

After review

  • The changelog item has been pasted to the CHANGELOG.md.

Comments

  • This PR adds automation for handling PR metadata. Further updates may be required based on additional fields or integrations.

@MisRob
Copy link
Member

MisRob commented Oct 2, 2024

@rtibbles companion PR learningequality/.github#1

@MisRob MisRob self-assigned this Oct 8, 2024
@MisRob MisRob merged commit aa45be3 into learningequality:main Oct 11, 2024
learning-equality-bot bot pushed a commit that referenced this pull request Oct 11, 2024
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.

2 participants