You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This rule should notify users that the newly added meta section must include a description key or the user will immediately receive a new lint warning from DescriptionMissing.
The text was updated successfully, but these errors were encountered:
This problem sparked an idea for "linking" rules. I imagine the output of sprocket explain MissingMetas saying You should also look at 'DescriptionMissing' (or something to that effect). This linkage would go in one direction. Presumably a user invoking sprocket explain DescriptionMissing has already written meta sections and shouldn't be linked back to MissingMetas.
The Rule trait could have a new method fn linked_rules() -> Vec<&'static str> where the Vec contains other Rule IDs.
The above text is a tangent and has been migrated to its own issue (#173). This issue should remain focused on clarifying explanation() text
We probably have other explanation() text that could use wordsmithing. Whoever gets around to rewording the MissingMetas rule, I'd encourage them to take a pass at our other lint rules too!
wdl/wdl-lint/src/rules/missing_metas.rs
Line 57 in cf378af
This rule should notify users that the newly added
meta
section must include adescription
key or the user will immediately receive a new lint warning fromDescriptionMissing
.The text was updated successfully, but these errors were encountered: