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

removed duplicate import of queueHandler and updated copyright message. #5842

Closed
wants to merge 1 commit into from

Conversation

derekriemer
Copy link
Collaborator

In source/browseMode.py there are duplicate imports.
import itertools
import collections
import winsound
import time
import weakref
import wx
import queueHandler
from logHandler import log
import review
import scriptHandler
import eventHandler
import nvwave
import queueHandler

removed the first import because it is a duplicate.

@@ -1 +1 @@
Subproject commit 239621041225ce15b93d6aeb96945eaf67e10898
Subproject commit a7f918c421e91f1b1601ab822ede2c869275ef5c
Copy link
Contributor

Choose a reason for hiding this comment

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

This PR updates miscDeps, but it shouldn't. Just something to be careful of for future commits.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oh. How did that happen?
Whoops. Does git pull modify anything because I have master set to pull
from origin/master here locally.

On 3/30/2016 9:15 PM, James Teh wrote:

In miscDeps
#5842 (comment):

@@ -1 +1 @@
-Subproject commit 239621041225ce15b93d6aeb96945eaf67e10898
+Subproject commit a7f918c421e91f1b1601ab822ede2c869275ef5c

This PR updates miscDeps, but it shouldn't. Just something to be
careful of for future commits.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/nvaccess/nvda/pull/5842/files/ae68ffa67b8deeae01c6d60f0c8050b4ec4d05b1#r57996394


Derek Riemer
  • Department of computer science, third year undergraduate student.
  • Proud user of the NVDA screen reader.
  • Open source enthusiast.
  • Member of Bridge Cu
  • Avid skiier.

Websites:
Honors portfolio http://derekriemer.com
Awesome little hand built weather app!
http://django.derekriemer.com/weather/

email me at derek.riemer@colorado.edu mailto:derek.riemer@colorado.edu
Phone: (303) 906-2194

Copy link
Contributor

@jcsteh jcsteh Mar 31, 2016 via email

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Cool. I won't use git add . in the future for simple things and that
should solve most of this. Did this pr get closed because I realized I
based it off of my master and if I push new committs there it will put
them here.

On 3/30/2016 9:19 PM, James Teh wrote:

In miscDeps
#5842 (comment):

@@ -1 +1 @@
-Subproject commit 239621041225ce15b93d6aeb96945eaf67e10898
+Subproject commit a7f918c421e91f1b1601ab822ede2c869275ef5c
You probably did git commit -a, added all files with git add or
something like that, but you had an outdated miscDeps submodule at the
time. Careful observance of git status and/or git show after the
commit will prevent such things.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/nvaccess/nvda/pull/5842/files/ae68ffa67b8deeae01c6d60f0c8050b4ec4d05b1#r57996555


Derek Riemer
  • Department of computer science, third year undergraduate student.
  • Proud user of the NVDA screen reader.
  • Open source enthusiast.
  • Member of Bridge Cu
  • Avid skiier.

Websites:
Honors portfolio http://derekriemer.com
Awesome little hand built weather app!
http://django.derekriemer.com/weather/

email me at derek.riemer@colorado.edu mailto:derek.riemer@colorado.edu
Phone: (303) 906-2194

@jcsteh jcsteh closed this in 78e3f10 Mar 31, 2016
@derekriemer
Copy link
Collaborator Author

Ignore reply to comment about miskkdeps.

On 3/30/2016 9:24 PM, James Teh wrote:

Closed #5842 #5842 via 78e3f10
78e3f10.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#5842 (comment)


Derek Riemer
  • Department of computer science, third year undergraduate student.
  • Proud user of the NVDA screen reader.
  • Open source enthusiast.
  • Member of Bridge Cu
  • Avid skiier.

Websites:
Honors portfolio http://derekriemer.com
Awesome little hand built weather app!
http://django.derekriemer.com/weather/

email me at derek.riemer@colorado.edu mailto:derek.riemer@colorado.edu
Phone: (303) 906-2194

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

Successfully merging this pull request may close these issues.

None yet

2 participants