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

Create stats_lambda.yml #6570

Closed
wants to merge 51 commits into from
Closed
Changes from 1 commit
Commits
Show all changes
51 commits
Select commit Hold shift + click to select a range
54b215a
Create stats_lambda.yml
brmur Jun 20, 2024
834c27a
Merge branch 'main' into stats_lambda
beqqrry-aws Jun 20, 2024
0186620
Update stats_lambda.yml
brmur Jun 21, 2024
f6a5289
Update stats_lambda.yml
brmur Jun 21, 2024
1b7983a
Update stats_lambda.yml
brmur Jun 21, 2024
80f5efe
Update stats_lambda.yml
brmur Jun 21, 2024
a1ae60f
Merge pull request #1 from brmur/stats_lambda
brmur Jun 21, 2024
4ab4c64
Update stats_lambda.yml
brmur Jun 21, 2024
19e9295
Merge pull request #2 from brmur/main
brmur Jun 21, 2024
3d39bbe
Update stats_lambda.yml
brmur Jun 21, 2024
ea20979
Merge pull request #3 from brmur/main
brmur Jun 21, 2024
1aba9d1
Update stats_lambda.yml
brmur Jun 21, 2024
4866c46
Merge pull request #4 from brmur/stats_lambda
brmur Jun 21, 2024
5dad632
Update stats_lambda.yml
brmur Jun 21, 2024
0ada20c
Merge pull request #5 from brmur/main
brmur Jun 21, 2024
b244bac
Update stats_lambda.yml
brmur Jun 21, 2024
5e523ed
Update stats_lambda.yml
brmur Jun 21, 2024
72130a3
Update stats_lambda.yml
brmur Jun 21, 2024
3e43e4c
Update stats_lambda.yml
brmur Jun 21, 2024
0531bca
Merge pull request #6 from brmur/main
brmur Jun 21, 2024
274b165
Update stats_lambda.yml
brmur Jun 21, 2024
c4be055
Merge pull request #7 from brmur/main
brmur Jun 21, 2024
1878a5f
Update stats_lambda.yml
brmur Jun 21, 2024
60a63a5
Merge pull request #8 from brmur/stats_lambda
brmur Jun 21, 2024
65da411
Update stats_lambda.yml
brmur Jun 21, 2024
8b9c645
Merge pull request #9 from brmur/main
brmur Jun 21, 2024
223ad36
Update stats_lambda.yml
brmur Jun 21, 2024
99f857b
Merge pull request #10 from brmur/main
brmur Jun 21, 2024
22bc74d
Update stats_lambda.yml
brmur Jun 21, 2024
7358a4d
Merge pull request #11 from brmur/main
brmur Jun 21, 2024
9ec8cfa
Update stats_lambda.yml
brmur Jun 21, 2024
f831dfc
Update stats_lambda.yml
brmur Jun 21, 2024
8063a34
Update stats_lambda.yml
brmur Jun 21, 2024
0f3ec0b
Update stats_lambda.yml
brmur Jun 21, 2024
86532c2
Update stats_lambda.yml
brmur Jun 21, 2024
a0902ae
Merge pull request #13 from brmur/main
brmur Jun 21, 2024
79eb82c
Update stats_lambda.yml
brmur Jun 21, 2024
ff40a76
Merge pull request #14 from brmur/main
brmur Jun 21, 2024
3014f4c
Update stats_lambda.yml
brmur Jun 21, 2024
4fff82f
Update stats_lambda.yml
brmur Jun 21, 2024
11f06bb
Merge pull request #15 from brmur/main
brmur Jun 21, 2024
a0515f1
Update stats_lambda.yml
brmur Jun 21, 2024
ec10607
Merge pull request #16 from brmur/main
brmur Jun 21, 2024
3e07e45
Update stats_lambda.yml
brmur Jun 21, 2024
bed530a
Merge pull request #17 from brmur/main
brmur Jun 21, 2024
3e62e75
Update stats_lambda.yml
brmur Jun 21, 2024
e0f36ba
Merge pull request #18 from brmur/main
brmur Jun 21, 2024
cbe2064
Update stats_lambda.yml
brmur Jun 21, 2024
0994000
Merge pull request #19 from brmur/main
brmur Jun 21, 2024
60bb134
Update stats_lambda.yml
brmur Jun 21, 2024
ee79544
Merge pull request #20 from brmur/main
brmur Jun 21, 2024
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
31 changes: 31 additions & 0 deletions .github/workflows/stats_lambda.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
# This workflow executes a Lamdba function that recording metrics for each successfully merged PR.
brmur marked this conversation as resolved.
Show resolved Hide resolved
#
# You can adjust the behavior by modifying this file.
brmur marked this conversation as resolved.
Show resolved Hide resolved
# For more information, see:
# https://github.com/marketplace/actions/invoke-aws-lambda

name: Invoke runTheNumbers lambda functions
brmur marked this conversation as resolved.
Show resolved Hide resolved
on:
Copy link
Contributor

Choose a reason for hiding this comment

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

this line should be:
on: # yamllint disable-line rule:truthy
(this is new since you created this PR)

pull_request:
types:
- closed
workflow_dispatch:
env:
COMMIT: ${{github.event.pull_request.head.sha}}
jobs:
if_merged:
brmur marked this conversation as resolved.
Show resolved Hide resolved
if: github.event.pull_request.merged == true
runs-on: ubuntu-latest
steps:
brmur marked this conversation as resolved.
Show resolved Hide resolved
- uses: actions/checkout@v2
- run: echo "JSON_PAYLOAD={\"commit\" :\"${{ env.COMMIT }}\"}" >> $GITHUB_ENV
- name: Invoke runTheNumbers
uses: gagoar/invoke-aws-lambda@master
with:
AWS_ACCESS_KEY_ID: ${{ secrets.AWS_ACCESS_KEY_ID }}
AWS_SECRET_ACCESS_KEY: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
REGION: 'eu-west-1'
FunctionName: runTheNumbers
Payload: ${{env.JSON_PAYLOAD}}
InvocationType: RequestResponse
LogType: None
Copy link
Contributor

Choose a reason for hiding this comment

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

The whole file has inconsistent spacing. This is getting missed by the linter (separate issue) but each indent should be 2 spaces.

Copy link
Contributor

Choose a reason for hiding this comment

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

Linter has been updated and you can see the spacing line errors in the action details. The truthy one I covered on the comment for line 8.

Loading