Skip to content

Commit

Permalink
Merge pull request #269 from MalikIdreesHasanKhan/main
Browse files Browse the repository at this point in the history
Fixed some typos.
  • Loading branch information
rongxin-liu authored Jan 10, 2022
2 parents 2559484 + 558ce91 commit 958c27a
Show file tree
Hide file tree
Showing 6 changed files with 10 additions and 10 deletions.
4 changes: 2 additions & 2 deletions check50/_api.py
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,7 @@ class run:
:type env: dict
By default, the command will be run using the same environment as ``check50``,
these mappings may be overriden via the ``env`` parameter::
these mappings may be overridden via the ``env`` parameter::
check50.run("./foo").stdin("foo").stdout("bar").exit(0)
check50.run("./foo", env={ "HOME": "/" }).stdin("foo").stdout("bar").exit(0)
Expand Down Expand Up @@ -475,7 +475,7 @@ def hidden(failure_rationale):
:param failure_rationale: the rationale that will be displayed to the student if the check fails
:type failure_rationale: str
Exaple usage::
Example usage::
@check50.check()
@check50.hidden("Your program isn't returning the expected result. Try running it on some sample inputs.")
Expand Down
2 changes: 1 addition & 1 deletion check50/c.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ def compile(*files, exe_name=None, cc=CC, max_log_lines=50, **cflags):
check50.c.compile("foo.c", "bar.c", lcs50=True) # clang foo.c bar.c -o foo -std=c11 -ggdb -lm -lcs50
In the same vein, the default CFLAGS may be overriden via keyword arguments::
In the same vein, the default CFLAGS may be overridden via keyword arguments::
check50.c.compile("foo.c", "bar.c", std="c99", lm=False) # clang foo.c bar.c -o foo -std=c99 -ggdb
"""
Expand Down
2 changes: 1 addition & 1 deletion check50/flask.py
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ def content(self, output=None, str_output=None, **kwargs):
raise Failure(_("expected request to return HTML, but it returned {}").format(
self.response.mimetype))

# TODO: Remove once beautiful soup updates to accomodate python 3.7
# TODO: Remove once beautiful soup updates to accommodate python 3.7
with warnings.catch_warnings():
warnings.filterwarnings("ignore", category=DeprecationWarning)
content = BeautifulSoup(self.response.data, "html.parser")
Expand Down
8 changes: 4 additions & 4 deletions check50/runner.py
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ def check(dependency=None, timeout=60, max_log_lines=100):
This is particularly useful when writing e.g., a ``compiles`` check that compiles a
student's program (and checks that it compiled successfully). Any checks that run the
student's program will logically depend on this check, and since they inherit the
resulting filesystem of the check, they will immidiately have access to the compiled
resulting filesystem of the check, they will immediately have access to the compiled
program without needing to recompile.
Example usage::
Expand Down Expand Up @@ -235,7 +235,7 @@ def build_subgraph(self, targets):


def dependencies_of(self, targets):
"""Get all unique dependencies of the targetted checks (tartgets)."""
"""Get all unique dependencies of the targeted checks (tartgets)."""
inverse_graph = self._create_inverse_dependency_graph()
deps = set()
for target in targets:
Expand Down Expand Up @@ -283,7 +283,7 @@ def __enter__(self):
self._cd_manager = lib50.cd(internal.run_root_dir)
self._cd_manager.__enter__()

# TODO: Naming the module "checks" is arbitray. Better name?
# TODO: Naming the module "checks" is arbitrary. Better name?
self.checks_spec = importlib.util.spec_from_file_location("checks", self.checks_path)

# Clear check_names, import module, then save check_names. Not thread safe.
Expand Down Expand Up @@ -348,7 +348,7 @@ def _store_attributes(self):
in case multiprocessing is using spawn as its starting method.
"""

# Attributes only need to be passed explicitely to child processes when using spawn
# Attributes only need to be passed explicitly to child processes when using spawn
if multiprocessing.get_start_method() != "spawn":
return

Expand Down
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,6 @@
"console_scripts": ["check50=check50.__main__:main"]
},
url="https://github.com/cs50/check50",
version="3.3.4",
version="3.3.5",
include_package_data=True
)
2 changes: 1 addition & 1 deletion tests/runner_tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ def setUp(self):

# Keep track of get_start_method
# This function gets monkey patched to ensure run_check is aware of the multiprocessing context,
# without needing to explicitely pass the context to run_check.
# without needing to explicitly pass the context to run_check.
# The same behavior can't be achieved by multiprocessing.set_start_method as that can only run once per program
# https://docs.python.org/3/library/multiprocessing.html#contexts-and-start-methods
self._get_start_method = multiprocessing.get_start_method()
Expand Down

0 comments on commit 958c27a

Please sign in to comment.