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

Refactor #1

Merged
merged 7 commits into from
Dec 30, 2023
Merged

Refactor #1

merged 7 commits into from
Dec 30, 2023

Conversation

franzs
Copy link
Contributor

@franzs franzs commented Dec 29, 2023

Thanks for your work.

There is room for improvement of your script though. 🤓 I commited some suggestions to remove copy'n paste code. Feel free to use them at your will.

@daboynb
Copy link
Owner

daboynb commented Dec 29, 2023

Hi, thank you, currently I'm out of home for vacation and I don't have a PC. I will take a look when I'll be back ;)

@daboynb
Copy link
Owner

daboynb commented Dec 30, 2023

Screenshot_20231230-130437_Brave.png

With those changes the script thinks that the json file is always different, it's a pain to look from a phone, I will investigate with a pc😂

@franzs
Copy link
Contributor Author

franzs commented Dec 30, 2023

No worries. It's not urgent.

It seems there is an empty pif.json as the previous version in your screenshot. Do you have a link to the corresponding workflow run? Then I can have a look at it.

BTW. there are a lot of empty commits. They are created every five minutes and trigger a workflow. Do have an idea where they are from?

@daboynb
Copy link
Owner

daboynb commented Dec 30, 2023

Now it works, idk why ahaha

The empty commits are triggered by me to make the workflow run every 5 min, cron is delayed on GitHub actions.

@daboynb daboynb merged commit c199c7d into daboynb:main Dec 30, 2023
@franzs
Copy link
Contributor Author

franzs commented Dec 30, 2023

Thanks for merging. 😃

Let's talk about the empty commits and delayed cron schedules somewhere else. I'll create an issue for this... 🤓

@franzs franzs mentioned this pull request Dec 30, 2023
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.

None yet

2 participants