gauge-sh / tach

A Python tool to enforce dependencies, using modular architecture 🌎 Open source 🐍 Installable via pip πŸ”§ Able to be adopted incrementally - ⚑ Implemented with no runtime impact ♾️ Interoperable with your existing systems πŸ¦€ Written in rust
https://gauge.sh
MIT License
1k stars 35 forks source link

--web leads to an empty page #205

Closed timgl closed 2 weeks ago

timgl commented 1 month ago

I was trying Gauge on https://github.com/posthog/posthog, and ran gauge show --web, but the page it leads to is empty. https://show.gauge.sh/?uid=6eb4ed80-76cc-43fc-941e-f0238848a221

emdoyle commented 1 month ago

Hi @timgl ! Thanks for raising the issue, and trying out Tach.

This usually means that there are no dependencies listed in your tach.yml, which could mean the source root isn't where Tach expects, or you just need to run tach sync.

I pulled down the Posthog repo, left the source root as-is, and marked ./posthog and all its children as modules: image then ran tach sync, and tach show --web to generate this: https://show.gauge.sh/?uid=b908bdbb-0cea-453d-b414-191c823dbc64

Unfortunately it's pretty tough to read the resulting graph, we'll have to improve that for larger repos! In the meantime, tach show (without --web) will give you the underlying graphviz DOT files if you have another viewer you can use.

The tach.yml content itself might be more legible, and can give you a feel for your dependencies. Also check out tach report!

caelean commented 1 month ago

This is a good call out as well - if all of your modules are isolated right now we don't show anything, whereas we should show those modules that are fully isolated. Just updated show --web to do this! Example below -

image

In general there's definitely a lot we can do to make tach more ergonomic on larger repos - @timgl would love to pick your brain if you're open to it! Was awesome to hear from you at the YC OSS meetup on Friday πŸ˜„ If you're still in SF, β˜•/🍺 on us!

emdoyle commented 2 weeks ago

Closing this due to inactivity. Taking it as feedback that show could use some love.