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

Propose DAST entry #8

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Propose DAST entry #8

wants to merge 2 commits into from

Conversation

david-a-wheeler
Copy link
Contributor

DAST is a pain, because people agree on the expansion but not what it means. Let's at least make it clear what people agree on and what they don't.

DAST is a pain, because people agree on the expansion
but not what it means. Let's at least make it clear
what people agree on and what they don't.

Signed-off-by: David A. Wheeler <[email protected]>
@GeauxJD
Copy link
Contributor

GeauxJD commented Jan 17, 2025

@david-a-wheeler Need to add a couple of words to the wordlist for spellcheck. But also I created this PR to hopefully filter it out from checking text in URLs as words #9

Satisfy our spelling checker.

There weren't any *real* spelling errors, but we want to make the
checker happy.

The original quote said "webenabled" but I think it's okay to silently
add a hyphen. For the rest, add the words to the wordlist.

Signed-off-by: David A. Wheeler <[email protected]>
@david-a-wheeler
Copy link
Contributor Author

All fixed. Okay to merge? Hopefully this makes the process clear.

I intentionally chose a challenging word. We don't need glossaries to define the word "the", we need glossaries when there is confusion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants