Skip to content

Add and Update Documentation#169

Merged
PSNAppz merged 2 commits intoDedSecInside:devfrom
Jacobite747:dev
Oct 5, 2019
Merged

Add and Update Documentation#169
PSNAppz merged 2 commits intoDedSecInside:devfrom
Jacobite747:dev

Conversation

@Jacobite747
Copy link
Contributor

Issue #97

Changes Proposed

  • Modifed some existing documentation
  • Added further documentation on top of what was already present.

Explanation of Changes

Generally good practice to have fully documented code; aids in the process of creating comprehensive and usable user documentation and API structure, as well as for future developers.

Transcript of new feature/change

Documentation Coverage Check Gist

robots.add(url)
cprint("Robots.txt found",'blue')
print(robots)
print(robots)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

undefined name 'robots'

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Product of the structure of the code I think, same below.

if '*' not in match:
url = main_url + match
robots.add(url)
robots.add(url)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

undefined name 'robots'

Copy link
Member

@PSNAppz PSNAppz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks neat. :)
Excellent work @Jacobite747.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants