Skip to content

Replace python rt with zig#164

Merged
DrDeano merged 1 commit intodevelopfrom
feature/replace-python-rt-with-zig
Jun 23, 2020
Merged

Replace python rt with zig#164
DrDeano merged 1 commit intodevelopfrom
feature/replace-python-rt-with-zig

Conversation

@DrDeano
Copy link
Member

@DrDeano DrDeano commented Jun 13, 2020

Part of #117

@DrDeano DrDeano requested a review from SamTebbs33 June 13, 2020 11:30
Copy link
Collaborator

@SamTebbs33 SamTebbs33 left a comment

Choose a reason for hiding this comment

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

Looking good

Copy link
Collaborator

@SamTebbs33 SamTebbs33 left a comment

Choose a reason for hiding this comment

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

Free to merge once it passes the tests

@DrDeano DrDeano force-pushed the feature/replace-python-rt-with-zig branch from ed5dfed to 4d9c07e Compare June 22, 2020 19:27
Added the new testing to the OS files

Some spelling
INOUT => IN/OUT
Added some doc comments to log

Added the new runtime to the build + added the None test mode

Moved some stuff around
None test mode is the default to run/build the OS normally with no runtime tests.

Add the new runtime testing to the CI


Updated README and CI


Increased timeout


Print the log message


Spelling


Move runtime to test folder


Add new RT to tty


Add a log to use the unmapped memory to cause page fault in release


Ensure the integer overflow happens even in release builds
@DrDeano DrDeano force-pushed the feature/replace-python-rt-with-zig branch from a040577 to 2c91e6f Compare June 23, 2020 11:44
@DrDeano DrDeano merged commit 10bf243 into develop Jun 23, 2020
@DrDeano DrDeano deleted the feature/replace-python-rt-with-zig branch June 23, 2020 11:47
@SamTebbs33 SamTebbs33 mentioned this pull request Jul 20, 2020
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.

2 participants