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

local html reports improved #592

Merged
merged 8 commits into from
Sep 24, 2022
Merged

local html reports improved #592

merged 8 commits into from
Sep 24, 2022

Conversation

itsdivyanshjain
Copy link
Collaborator

Checklist

  • I have followed the Contributor Guidelines.
  • The code has been thoroughly tested in my local development environment with flake8 and pylint.
  • The code is Python 3 compatible.
  • The code follows the PEP8 styling guidelines with 4 spaces indentation.
  • This Pull Request relates to only one issue or only one feature
  • I have referenced the corresponding issue number in my commit message
  • I have added the relevant documentation.
  • My branch is up-to-date with the Upstream master branch.

Changes proposed in this pull request

Improved html local reports

Your development environment

  • OS: x
  • OS Version: x
  • Python Version: x

@Ali-Razmjoo
Copy link
Collaborator

The only issue I saw was when I tried to copy the JSON content; the Unicode came with it. Maybe there should be a copy to the clipboard button.

▼{
    "timeout": 3,
    "headers": ▼{
        "User-Agent": "Nettacker 0.0.3 TRES"
    },
    "allow_redirects": false,
    "url": "http://newsroom.uber.com:80/wp-content/plugins/akismet/readme.txt",
    "method": "get",
    "response": ▼{
        "conditions_results": ▼{
            "content": ▼[
                "Tags:"
            ],
            "status_code": ▼[
                "200"
            ]
        }
    }
}

@securestep9 securestep9 merged commit 02d56f1 into master Sep 24, 2022
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.

3 participants