Skip to content

Add test action #240

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

Merged
merged 2 commits into from
Mar 14, 2025
Merged

Add test action #240

merged 2 commits into from
Mar 14, 2025

Conversation

oguzhanunlu
Copy link
Collaborator

@oguzhanunlu oguzhanunlu commented Mar 13, 2025

This PR adds a test action to prepare a json with dated name under resources as well as updating the latest json.

@oguzhanunlu oguzhanunlu self-assigned this Mar 13, 2025
@oguzhanunlu
Copy link
Collaborator Author

Test action run successfully at https://github.com/snowplow-referer-parser/referer-parser/actions/runs/13855805169/job/38772241342 and added a commit with a clear message at 27b1bbc containing a test json with a datetimed name and a latest json.

git config --local user.name "github-actions[bot]"
git add resources/referers-$datetime.json resources/referers-latest.json
git commit -m "Add referers-$datetime.json"
git push
Copy link
Collaborator

Choose a reason for hiding this comment

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

Does this push to a branch? And if yes, which branch?

Can you give an example of the URL that we then use for testing in the pipeline?

Copy link
Collaborator Author

@oguzhanunlu oguzhanunlu Mar 14, 2025

Choose a reason for hiding this comment

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

this pushes to referer-json branch, pointed at

ref: referer-json

an example url that I used in my test is

https://raw.githubusercontent.com/snowplow-referer-parser/referer-parser/refs/heads/referer-json/resources/referers-2025-03-14-113945.json

Copy link
Collaborator

Choose a reason for hiding this comment

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

OK understood. I like that.

Are you also going to delete the pull_request.yml github action, which I think you only added as a test/demo?

Copy link
Collaborator

Choose a reason for hiding this comment

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

And sorry one more thing... please could you add a code comment to this github action, to remind us what url to look for.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Are you also going to delete the pull_request.yml github action, which I think you only added as a test/demo?

I wasn't planning to delete it as it makes sure any change to referers.yml in a PR is valid and the file can still be converted to json.

I'll add a comment to the github action.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@oguzhanunlu oguzhanunlu requested a review from istreeter March 14, 2025 13:22
@istreeter istreeter merged commit 65b7cf8 into master Mar 14, 2025
2 checks passed
@istreeter istreeter deleted the test-action branch March 14, 2025 13:29
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