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

Make bindata.go optional, remove from repository #78

Closed
wants to merge 6 commits into from

Conversation

strk
Copy link
Member

@strk strk commented Nov 5, 2016

By default will read assets from filesystems, but can embed them
by passing the "bindata" build tag (go build -tags bindata)

Fixes #77

By default will read assets from filesystems, but can embed them
by passing the "bindata" build tag (go build -tags bindata)

Fixes gogs#77
@strk strk added type/enhancement An improvement of existing functionality pr/wip This PR is not ready for review labels Nov 5, 2016
@strk
Copy link
Member Author

strk commented Nov 5, 2016

TODO:

  • Cache read files and directories
  • Check if filenames need or not be canonicalized (moot point, I dubt anyone develops on windows)

@strk
Copy link
Member Author

strk commented Nov 5, 2016

@tboerger if you're ahead of this feel free to discard it, or help is welcome :)

@tboerger
Copy link
Member

tboerger commented Nov 5, 2016

I already said that I'm working on that...

@strk
Copy link
Member Author

strk commented Nov 5, 2016

Will it show up in #74, then ?

@codecov-io
Copy link

codecov-io commented Nov 5, 2016

Current coverage is 2.43% (diff: 31.25%)

Merging #78 into master will increase coverage by 0.24%

@@            master       #78   diff @@
========================================
  Files           31        32     +1   
  Lines         7508      7572    +64   
  Methods          0         0          
  Messages         0         0          
  Branches         0         0          
========================================
+ Hits           164       184    +20   
- Misses        7327      7368    +41   
- Partials        17        20     +3   

Powered by Codecov. Last update b58b634...0aecf85

@strk strk removed the pr/wip This PR is not ready for review label Nov 5, 2016
@strk
Copy link
Member Author

strk commented Nov 5, 2016

Caching was pushed, ready for review (or test if anyone is really on Windows)

@tboerger
Copy link
Member

tboerger commented Nov 7, 2016

I would close that in favor of the ongoing work done by me as already asked in #78 (comment)

@tboerger tboerger closed this Nov 7, 2016
@tboerger tboerger added reviewed/invalid issue/duplicate The issue has already been reported. and removed type/enhancement An improvement of existing functionality reviewed/invalid labels Nov 7, 2016
lunny pushed a commit to lunny/gitea that referenced this pull request Feb 7, 2019
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue/duplicate The issue has already been reported.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Externalize bindata.go to an auto-generated file
3 participants