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

dexcom_one_plus/main_dexcomoneplus.patch #136

Closed
wants to merge 14 commits into from

Conversation

VladStanciu
Copy link

hello , why the dexcom_one_plus/main_dexcomoneplus.patch is not working ?

@VladStanciu VladStanciu reopened this Jun 12, 2024
@bjornoleh
Copy link
Contributor

bjornoleh commented Jun 13, 2024

hello , why the dexcom_one_plus/main_dexcomoneplus.patch is not working ?

Hi, first of all, I don’t think you intended to open a pull request to LoopKit/LoopWorkspace? This is for suggesting code changes to Loop, that was probably what you meant to do here. You can close this pull request.

Your workflow is failing due to not having removed the # from the beginning of line be 10 in your workflow.

You have to remove the # from both of these lines if you want to enable scheduled runs:

months
#schedule:
#- cron: '0 17 1 */2 *' # Runs at 17:00 UTC on the 1st in Jan, Mar, May, Jul, Sep and Nov.

You will find the specific error details from the links below:

https://github.com/VladStanciu/LoopWorkspace/actions/runs/9489356106

https://github.com/VladStanciu/LoopWorkspace/actions/runs/9489356106/workflow

But from your logs it looks like you already figured this out :-)

The mystery to me is how the GitHub action appears to have been started in LoopKit/LoopWorkspace upon your pull request :-) Apparently, GitHub checks the workflow files when a PR is submitted.

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