-
Notifications
You must be signed in to change notification settings - Fork 56
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
Substitute illegal characters in filename to hyphen #479
Changes from all commits
a7b3b34
9b1f727
78ec797
ac22c1d
4aed20b
72a5349
9638c74
f028eac
f21f370
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -17,6 +17,7 @@ | |||||||||||||||||
import logging | ||||||||||||||||||
import re | ||||||||||||||||||
import socket | ||||||||||||||||||
import unicodedata | ||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. F401: 'unicodedata' imported but unused ℹ️ Expand to see all @sonatype-lift commandsYou can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
Note: When talking to LiftBot, you need to refresh the page to see its response. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. vulture-90: unused import 'unicodedata' ℹ️ Expand to see all @sonatype-lift commandsYou can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
Note: When talking to LiftBot, you need to refresh the page to see its response. |
||||||||||||||||||
import xml.etree.cElementTree as ET | ||||||||||||||||||
|
||||||||||||||||||
import redis | ||||||||||||||||||
|
@@ -367,6 +368,10 @@ def postToGithub(message, encodedContent, filename): | |||||||||||||||||
return r.status_code, r.json() | ||||||||||||||||||
|
||||||||||||||||||
|
||||||||||||||||||
def removeSpecialCharacters(filename): | ||||||||||||||||||
return re.sub(r'[#%&{}<>*?/$!\'":@+`|=]', "-", filename) | ||||||||||||||||||
|
||||||||||||||||||
|
||||||||||||||||||
def parseXmlString(xmlString): | ||||||||||||||||||
""" View for generating a spdx license xml | ||||||||||||||||||
returns a dictionary with the xmlString license fields values | ||||||||||||||||||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
E501: line too long (86 > 79 characters)
❗❗ 3 similar findings have been found in this PR
🔎 Expand here to view all instances of this finding
Visit the Lift Web Console to find more details in your report.
ℹ️ Expand to see all @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
@sonatype-lift ignore
@sonatype-lift ignoreall
@sonatype-lift exclude <file|issue|path|tool>
file|issue|path|tool
from Lift findings by updating your config.toml fileNote: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.