Skip to content

Commit

Permalink
changes variable name from f -> file
Browse files Browse the repository at this point in the history
- addresses the following pylint error:
C0103: Variable name f doesn't conform to snake_case naming style (invalid-name)
  • Loading branch information
shailshouryya committed May 4, 2020
1 parent 51f13db commit 0f514a3
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions yt_videos_list/selenium_macos.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,6 @@ def browser_exists(browser):
return browser in subprocess.getoutput('ls /Applications')

def get_browser_version(browser):
with open (f'/Applications/{browser}.app/Contents/Info.plist') as f:
info_plist = f.read()
with open (f'/Applications/{browser}.app/Contents/Info.plist') as file:
info_plist = file.read()
return re.search(r'<key>CFBundleShortVersionString</key>\s*<string>([0-9\.]+)', info_plist)[1]
4 changes: 2 additions & 2 deletions yt_videos_list/selenium_windows.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,8 @@ def get_firefox_version():
def get_opera_version():
drive = get_drive_letter()
user = get_user_name()
with open(rf'{drive}:\Users\{user}\AppData\Local\Programs\Opera\installation_status.json', 'r') as f:
opera = json.load(f)
with open(rf'{drive}:\Users\{user}\AppData\Local\Programs\Opera\installation_status.json', 'r') as file:
opera = json.load(file)
return opera['_subfolder']

def get_chrome_version():
Expand Down

0 comments on commit 0f514a3

Please sign in to comment.