Skip to content
This repository has been archived by the owner on Mar 25, 2024. It is now read-only.

Only support 64 bit architecture #101

Open
kealan opened this issue Feb 27, 2023 · 3 comments
Open

Only support 64 bit architecture #101

kealan opened this issue Feb 27, 2023 · 3 comments

Comments

@kealan
Copy link
Contributor

kealan commented Feb 27, 2023

Goal

In order to simplify the project it is proposed to only build and test
for 64 bit architectures. The code that refers to 32 bit and 16 bit will
be removed.

Resources

Acceptance criteria

  1. Code is commented
  2. Tests pass with more than 80% test coverage
  3. Static analysis should pass
  4. Code should be reviewed by two engineers
@giorgiozoppi
Copy link
Contributor

@kealan commented code in the repo is bad pratice. I would rather prefer to remove the code that it's not related. I would rather tag (i.e. LAST_SUPPORT_32_BIT) and then remove all the code will no longer supported. Today i am in Dublin at we work docklands, if you wanna meet and test.

@kealan
Copy link
Contributor Author

kealan commented Feb 27, 2023

Hi Giorgio That is a fair point. I will just remove the code.

@giorgiozoppi
Copy link
Contributor

giorgiozoppi commented Mar 26, 2023

Have you closed this? Is there any PR? How can I help here?

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

No branches or pull requests

2 participants