Skip to content
This repository has been archived by the owner on Feb 24, 2024. It is now read-only.

split out a few things from the generated action.App() implementation. #1008

Merged
merged 3 commits into from
Apr 5, 2018

Conversation

markbates
Copy link
Member

@markbates markbates commented Apr 5, 2018

Hopefully this will help with a bit more in-line documentation, as well, as showing a pattern for keeping that method tame when it starts getting big.

Hopefully this will help with a bit more in-line documentation, as well,
as showing a pattern for keeping that method tame when it starts getting
big.
@markbates markbates changed the title split out a few things from the generated action.App() implementation. Hopefully this will help with a bit more in-line documentation, as well, as showing a pattern for keeping that method tame when it starts getting big. split out a few things from the generated action.App() implementation. Apr 5, 2018
@markbates markbates added this to the 0.11.1 milestone Apr 5, 2018
@markbates markbates requested a review from stanislas-m April 5, 2018 20:51
@markbates markbates merged commit 0aa623c into development Apr 5, 2018
@markbates markbates deleted the extract-methods-in-gend-app-files branch April 5, 2018 22:32
stanislas-m pushed a commit that referenced this pull request May 12, 2018
#1008)

* split out a few things from the generated action.App() implementation.
Hopefully this will help with a bit more in-line documentation, as well,
as showing a pattern for keeping that method tame when it starts getting
big.

* updated to mention using a proxy

* added a little more documentation to the generated main.go
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants