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

Preload the Env variable, saves taking a lock. #5

Merged
merged 1 commit into from
Oct 8, 2022

Conversation

AtomicNibble
Copy link
Contributor

Preload the Env variable, saves taking a lock.

@sio4
Copy link
Member

sio4 commented Jul 23, 2022

Thank you for the PRI

It is considerable but need to check the actual flow in action (along with the generated app and envy)

@sio4 sio4 added the s: triage label Jul 23, 2022
@github-actions
Copy link

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment. Otherwise, this will be closed in 7 days.

@github-actions github-actions bot added the stale label Sep 27, 2022
@sio4 sio4 removed the stale label Sep 27, 2022
@sio4
Copy link
Member

sio4 commented Oct 8, 2022

Hi @AtomicNibble

Thanks for the PR again. The initiative of this PR is great, I and want to improve it some more. Since the PR's age is more than two years, I am not sure if you still have an interest in this PR. I am going to merge it first and will fix it some more.

Thanks!

csrf.go Show resolved Hide resolved
@sio4 sio4 merged commit 753703e into gobuffalo:main Oct 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants