Conversation
An update to PR Update fides_module.md documentation
JiwaniZakir
left a comment
There was a problem hiding this comment.
The removal of the ## Installation section from docs/fides_module.md eliminates the only place that explicitly mentioned the --use_fides=True flag and its behavioral caveat (that it's ignored when Slips runs on a file rather than an interface). While the docker pull/run commands appear redundant with the block already present at the top of the file, the note about interface-only operation carries important information that isn't captured anywhere else in this document. Before removing it entirely, consider folding that caveat into the ### Configuration section or the existing docker run example at the top, so users aren't left without guidance on the interface-only restriction. Additionally, the original note contained a typo ("lease" instead of "please"), which suggests the section may have been understated rather than truly redundant — worth preserving with a fix rather than a wholesale deletion.
An update to PR Update fides_module.md documentation
Check List (Check all the applicable boxes)