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

Fix hs.application.get() always returning nil #3216

Merged
merged 3 commits into from
Nov 2, 2022

Conversation

amlestin
Copy link
Contributor

@amlestin amlestin commented May 21, 2022

for #3206

pr-fix

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This pull request does not contain a valid label. Please add one of the following labels: pr-fix, pr-change, pr-feature, pr-maintenance

@amlestin amlestin marked this pull request as draft May 21, 2022 05:24
@amlestin amlestin marked this pull request as ready for review May 25, 2022 21:47
@amlestin amlestin closed this May 25, 2022
@amlestin amlestin reopened this May 25, 2022
@amlestin amlestin changed the title Fix app.get always returning nil Fix app.get always returning nil pr-fix May 25, 2022
@andmis
Copy link

andmis commented Jun 1, 2022

I'd love to see this get merged at some point!

@amlestin
Copy link
Contributor Author

amlestin commented Jun 1, 2022

I'd love to see this get merged at some point!

Do you know how I can add a pr-fix label to fix the workflow?

@andmis
Copy link

andmis commented Jun 1, 2022

I'd love to see this get merged at some point!

Do you know how I can add a pr-fix label to fix the workflow?

Nope, unfortunately I don't. I'm not a maintainer or anything, just a bystander / cheerleader!

@amlestin
Copy link
Contributor Author

amlestin commented Jun 1, 2022

I'd love to see this get merged at some point!

Do you know how I can add a pr-fix label to fix the workflow?

Nope, unfortunately I don't. I'm not a maintainer or anything, just a bystander / cheerleader!

Thanks anyway :) I think only the repo owner can edit it.

@latenitefilms
Copy link
Contributor

I believe only @asmagill and @cmsj have the power to assign labels.

@cmsj cmsj added the pr-fix Pull Request implementing a bug fix label Jun 1, 2022
@cmsj
Copy link
Member

cmsj commented Jun 1, 2022

Added the label and kicked the CI off, not that I would particularly expect the results to be relevant here (and don't worry if there are unrelated failures - some of our tests are just unreliable, unfortunately!)

@cmsj
Copy link
Member

cmsj commented Jun 1, 2022

Just one other thing: I'm quite short on time at the moment, but I'd like to dig into this a little deeper before merging, just to make sure we're not invalidating other assumptions somehow.

@github-actions
Copy link

github-actions bot commented Jun 1, 2022

View Test Results

342 tests   290 ✔️  11m 24s ⏱️
    2 suites    52 💤
    1 files        0

Results for commit b58d959.

♻️ This comment has been updated with latest results.

extensions/json/test_json.lua Outdated Show resolved Hide resolved
extensions/json/test_json.lua Outdated Show resolved Hide resolved
@cmsj
Copy link
Member

cmsj commented Jul 1, 2022

@amlestin this seems to include an unrelated change to the hs.json tests - was that intentional?

@amlestin
Copy link
Contributor Author

amlestin commented Jul 1, 2022

@cmsj cmsj merged commit 3889b25 into Hammerspoon:master Nov 2, 2022
@cmsj
Copy link
Member

cmsj commented Nov 2, 2022

Thanks, and sorry this has taken so long!

@cmsj cmsj changed the title Fix app.get always returning nil pr-fix Fix hs.application.get() always returning nil Mar 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-fix Pull Request implementing a bug fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants