sindresorhus / hide-files-on-github

Chrome extension - Hide nonessential files from the GitHub file browser
https://chrome.google.com/webstore/detail/hide-files-on-github/lpnakhpaodhdkleejaehlapdhbgjbddp
MIT License
320 stars 35 forks source link

Enterprise GitHub? #68

Closed kylebarron closed 5 years ago

kylebarron commented 6 years ago

Can this work with enterprise github as well?

sindresorhus commented 6 years ago

I could, yes, but not something I'm going to work on. I'll leave this open as "PR welcome. It could use the same approach as done here: https://github.com/sindresorhus/refined-github/blob/1e2428126e1e4eabe4e822eac6f2efd9263f02cc/source/background.js#L53-L55

hipstersmoothie commented 6 years ago

I'm gonna try to take a swing at this. It seems like a pretty big overhaul though. As I see it the steps would be:

- [ ] Emulate refined-github's Webpack/Babel Configuration - [ ] Convert options to https://github.com/bfred-it/webext-options-sync

fregante commented 6 years ago

Only the third point is necessary for this, all the mentioned modules can be used directly from manifest.json

hipstersmoothie commented 6 years ago

@bfred-it I tried doing it the non webpack way from your readmes and couldn't get it to work.

fregante commented 6 years ago

The error you mentioned on that repo was because it cannot find the file. They need to be in the same folder. The file is not node-specific

hipstersmoothie commented 6 years ago

I see your comment now. Would you advise on a build step that moves the files from node_modules to the extension directory? I would be hesitant to just move the files, since they could get out of date.

fregante commented 6 years ago

No need, I don’t update those modules nearly that often

hipstersmoothie commented 6 years ago

Ok! As long as @sindresorhus is ok with it I'll just move the files to the extension folder.

fregante commented 6 years ago

extension/libs would be best, that’s how they were set in another Sindre’s extension