summaryrefslogtreecommitdiff
path: root/CONTRIBUTION.md
AgeCommit message (Collapse)Author
2024-08-13Update build steps for MacOS (#4813)Jay Kwak
This commit adds a quick build step for MacOS on CONTRIBUTION.md. It is unclear what are the conditions that slang-llvm needs to be rebuilt locally. An issue is filed for it, #4812
2024-07-12Update CONTRIBUTION.md for path to slang-test.exe (#4619)Jay Kwak
The path to slang-test.exe has been changed recently and it needs to be updated on the document.
2024-07-10Update CONTRIBUTION.md after CI side changes (#4598)Jay Kwak
* Update CONTRIBUTION.md after CI side changes * Fix a typo --------- Co-authored-by: Yong He <yonghe@outlook.com>
2024-05-20This commit increases the minimum CMake version from 3.20 to 3.25. (#4193)Jay Kwak
I was trying to see if I can lower it to 3.16, but I found that we are currently using CMake feature that requires a version 3.25 not 3.20. This finding is not new. I made a similar change to CMakePresets.json a few days ago. At that time, I didn't realize that the same change had to be made for CMakeList.txt as well.
2024-05-17Add warning about CMake version on CONTRIBUTION.mdJay Kwak
Currently CMake version is required to be 3.20 or above. The version requirement is properly defined in our CMakeList.txt file. But older versions of CMake may not even print an error about the version requirement.
2024-05-13Update CONTRIBUTION.mdJay Kwak
Clarify which `slang.sln` file needs to be used for cmake workflow.
2024-04-24Updating CONTRIBUTION guide to use CMake (#4017)Jay Kwak
Releated to #3703 Removing the build instruction with Premake and replacing it with an instruction with CMake. It is because we are going to move over to CMake anytime soon. Bumping the required CMake version to 3.25.0. When CMakePresets.json has "version:6", it requires CMake version to be 3.25 or above. See the URL below for more information, https://cmake.org/cmake/help/latest/release/3.25.html CMakeLists.txt copies the prebuilt binary files from external/slang-binaries/bin/windows-x64 CMakeLists.txt was copying "slang-llvm.dll" to build/Release/lib directory when it should have been build/Release/bin. It made slang-test to ignore all FILECHECK tests. This is fixed. Co-authored-by: Yong He <yonghe@outlook.com>
2024-01-23Adding CONTRIBUTION.md (#3472)Jay Kwak
* Adding CONTRIBUTION.md Fixes #3372 Adding CONTRIBUTION.md that describes the workflow for the contributors at more details. * Add instructions for PR process in CONTRIBUTE.me This commit adds an instruction of how to handle the case when you got feedbacks during Pull Request process. * Fix a formatting problem for Code Style When a word is wrapped with "lessThen" and "greaterThan" characters, it may disappear on the result based on the formatting syntax of .MD file. They need to be wrapped with a single back-tick character to avoid the problem. --------- Co-authored-by: Yong He <yonghe@outlook.com>