Skip to content

Added small update to sample app#307

Closed
Thanus31 wants to merge 11 commits into
nisalgunawardhana:mainfrom
Thanus31:feature/my-feature
Closed

Added small update to sample app#307
Thanus31 wants to merge 11 commits into
nisalgunawardhana:mainfrom
Thanus31:feature/my-feature

Conversation

@Thanus31
Copy link
Copy Markdown

@Thanus31 Thanus31 commented May 8, 2026

Description

This pull request includes updates related to GitHub Actions learning tasks.

Changes Made

  • Added and tested custom GitHub Actions workflows
  • Configured environment variables and secrets
  • Implemented artifact upload/download functionality
  • Added conditional workflow execution
  • Updated workflow configuration files
  • Verified workflows using GitHub Actions logs

Fixes # (issue)


Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 📚 Documentation update
  • 🔧 Configuration change
  • ♻️ Code refactor
  • 🚀 Performance improvement
  • 💥 Breaking change

How Has This Been Tested?

  • Tested manually on local machine
  • Verified using GitHub Actions workflow runs

Test Configuration

  • Node.js version: 22.x
  • OS: Windows

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes generate no new warnings
  • New and existing workflows passed successfully

Screenshots or Workflow Logs

Workflow runs were verified successfully in the GitHub Actions tab.


Additional Context

This PR was created as part of GitHub Actions practice and learning activities.

Copilot AI review requested due to automatic review settings May 8, 2026 06:38
Copy link
Copy Markdown

Copilot AI left a comment

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 updates repository documentation and adjusts/adds GitHub Actions workflows used for the learning sample app and tasks.

Changes:

  • Adds minor README text to the root README and the sample app README.
  • Adds two new example workflows (custom.yml, conditional.yml) for learning exercises.
  • Significantly rewrites the existing build-test.yml workflow (triggers, matrix, steps, artifacts).

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
sample-app/README.md Adds a short trailing documentation line in the sample app README.
README.md Adds a short trailing tagline and extra blank lines at the end of the root README.
.github/workflows/hello-world.yml Adds a new comment line near the top of the workflow.
.github/workflows/custom.yml Introduces a new workflow that runs tests on pushes to develop, including env vars and a secrets-related step.
.github/workflows/conditional.yml Introduces a new workflow demonstrating conditional step execution based on branch.
.github/workflows/build-test.yml Replaces the prior build/test pipeline with a simpler develop-only workflow that runs tests, uploads an artifact, and downloads it in a dependent job.

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

Comment on lines 12 to +15
strategy:
matrix:
node-version: [16.x, 18.x]
node-version: [22.x]

branches: [main]
workflow_dispatch:
branches:
- develop
Comment on lines 12 to +15
strategy:
matrix:
node-version: [16.x, 18.x]
node-version: [22.x]

env:
TEST_SECRET: ${{ secrets.TEST_SECRET }}
run: |
echo "Secret loaded successfully"
Comment thread sample-app/README.md
Comment on lines +157 to 159
Updated sample app documentation

**Happy learning! 🎓**
Comment thread README.md
Comment on lines 610 to 614

Learning GitHub Actions 🚀



Comment on lines +1 to 7
name: Build Test Workflow

on:
push:
branches: [main, develop]
paths:
- 'sample-app/**'
- '.github/workflows/build-test.yml'
pull_request:
branches: [main]
workflow_dispatch:
branches:
- develop

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