Skip to content

Commit

Permalink
test using different container
Browse files Browse the repository at this point in the history
  • Loading branch information
ksentak committed Aug 13, 2024
1 parent 0f05734 commit d49e0d7
Show file tree
Hide file tree
Showing 3 changed files with 22 additions and 17 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/release-github.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,17 +19,17 @@ jobs:
name: Release
runs-on: ubuntu-latest
container:
image: kevinshahfws/node-c:3.0
image: ksentak01/cpp-ubuntu:latest
steps:
# Checkout/install project
- name: Checkout
uses: actions/checkout@v2
uses: actions/checkout@v4
with:
fetch-depth: 0
repo-token: ${{ secrets.SEMANTIC_RELEASE_BOT_PAT }}
persist-credentials: false
- name: Setup Node.js
uses: actions/setup-node@v2
uses: actions/setup-node@v4
with:
node-version: 'lts/*'
- name: Set up NPM token
Expand Down Expand Up @@ -94,7 +94,7 @@ jobs:
npm run dist
- name: Check if semantic-release created a build
id: check_build
uses: andstor/file-existence-action@v2
uses: andstor/file-existence-action@v3
with:
files: './src/sdks/core/dist/lib/firebolt.mjs'
- name: Get version from package.json
Expand Down
26 changes: 15 additions & 11 deletions .github/workflows/utils.sh
Original file line number Diff line number Diff line change
Expand Up @@ -189,17 +189,21 @@ function generateSource() {
exit 1
fi

# echo "Generate source for Discovery SDK"
# cd ../discovery
# npm run cpp

# if [ $? -eq 0 ]
# then
# echo "Native Discovery SDK generated successfully"
# else
# echo "Native Discovery SDK generation failed"
# exit 1
# fi
echo "Generate source for Discovery SDK"
cd ../../../../discovery
npm run cpp

if [ $? -eq 0 ]
then
echo "Native Discovery SDK generated successfully"
echo " Discovery SDK Location"
cd build/cpp/src/
ls -la
echo " ************ Source Generation Completed for Discovery SDK ************"
else
echo "Native Discovery SDK generation failed"
exit 1
fi
}

function cloneAndInstallThunder() {
Expand Down
5 changes: 3 additions & 2 deletions src/sdks/discovery/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
"native": "npx firebolt-openrpc sdk --input ./dist/firebolt-discovery-open-rpc.json --template ./src/js --output ./build/c/src --language ../../../node_modules/@firebolt-js/openrpc/languages/c",
"cpp": "npm run cpp:compile && npm run cpp:install",
"cpp:compile": "npx firebolt-openrpc sdk --input ./dist/firebolt-discovery-open-rpc.json --template ./src/cpp --output ./build/cpp/src --static-module Platform --language ../../../node_modules/@firebolt-js/openrpc/languages/cpp",
"cpp:install": "./build/cpp/src/scripts/install.sh -i ./build/cpp/src -s ./build/cpp/src/ -m discovery",
"cpp:install": "chmod +x ./build/cpp/src/scripts/install.sh && sdkVersion=$(npm run -s getVersion) && ./build/cpp/src/scripts/install.sh -i ./build/cpp/src -s ./build/cpp/src/ -m discovery -v $sdkVersion",
"compile": "cd ../../.. && npm run compile",
"slice": "npx firebolt-openrpc slice -i ../../../dist/firebolt-open-rpc.json --sdk ./sdk.config.json -o ./dist/firebolt-discovery-open-rpc.json",
"docs": "npx firebolt-openrpc docs --input ./dist/firebolt-discovery-open-rpc.json --output build/docs/markdown --as-path",
Expand All @@ -29,7 +29,8 @@
"test": "npm run test:setup && NODE_OPTIONS=--experimental-vm-modules npx --config=jest.config.json --detectOpenHandles jest",
"prepack": "node ../../js/version.mjs validate && npm run broilerplate",
"broilerplate": "rm ./CONTRIBUTING.md && cp ../../../CONTRIBUTING.md ./CONTRIBUTING.md && rm ./LICENSE && cp ../../../LICENSE ./LICENSE && rm ./NOTICE && cp ../../../NOTICE ./NOTICE",
"prettier": "prettier build/**/*.mjs --write --parser babel && prettier build/**/*.md --write --parser markdown"
"prettier": "prettier build/**/*.mjs --write --parser babel && prettier build/**/*.md --write --parser markdown",
"getVersion": "node -p \"require('./package.json').version\""
},
"repository": {
"type": "git",
Expand Down

0 comments on commit d49e0d7

Please sign in to comment.