Skip to content

Remove final incorrect instances of %AppData% / %LocalAppData% from Windows cache examples#762

Merged
vsvipul merged 1 commit intoactions:mainfrom
magnetikonline:remove-win-load-app-data
May 11, 2022
Merged

Remove final incorrect instances of %AppData% / %LocalAppData% from Windows cache examples#762
vsvipul merged 1 commit intoactions:mainfrom
magnetikonline:remove-win-load-app-data

Conversation

@magnetikonline
Copy link
Copy Markdown
Contributor

As noted by @vsvipul and related to #651 - dropping final mentions of %AppData% / %LocalAppData% from example docs.

@vsvipul
Copy link
Copy Markdown
Contributor

vsvipul commented Mar 10, 2022

Will wait for someone to test if this really makes any difference on windows. @jheysaav . If you want to test @magnetikonline can use https://github.com/duxtland/framework repo.

@magnetikonline
Copy link
Copy Markdown
Contributor Author

Will wait for someone to test if this really makes any difference on windows. @jheysaav . If you want to test @magnetikonline can use https://github.com/duxtland/framework repo.

Not a problem, makes sense. 👍

But as per my test repo in https://github.com/magnetikonline/golang-win-cache-test to confirm #651 (as suggested by @brcrista) - these %LocalAppData% / %AppData% substitutions don't actually work as expected.

So I'm pretty confident these are in the same vein.

@magnetikonline
Copy link
Copy Markdown
Contributor Author

Bump @vsvipul / @jheysaav

@vsvipul vsvipul self-assigned this May 11, 2022
Copy link
Copy Markdown
Contributor

@vsvipul vsvipul left a comment

Choose a reason for hiding this comment

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

lgtm

@vsvipul vsvipul merged commit 95f200e into actions:main May 11, 2022
@magnetikonline
Copy link
Copy Markdown
Contributor Author

Thanks @vsvipul

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