Skip to content

Commit

Permalink
Fixes codeclimate issues
Browse files Browse the repository at this point in the history
  • Loading branch information
prasadtalasila committed Oct 7, 2023
1 parent f0324ca commit 6d464da
Show file tree
Hide file tree
Showing 6 changed files with 16 additions and 16 deletions.
2 changes: 1 addition & 1 deletion deploy/single-script-install.sh
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ then
fi
NODE_MAJOR=18
printf "deb [signed-by=/etc/apt/keyrings/nodesource.gpg] \
https://deb.nodesource.com/node_$NODE_MAJOR.x nodistro main" | \
https://deb.nodesource.com/node_%s.x nodistro main" "$NODE_MAJOR" | \
sudo tee /etc/apt/sources.list.d/nodesource.list
sudo apt-get update
sudo apt-get install -y nodejs
Expand Down
10 changes: 4 additions & 6 deletions deploy/vagrant/make_boxes/dtaas/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,7 @@ This README provides instructions on creating a custom Operating System
virtual disk for running the DTaaS software. The virtual disk is managed
by **vagrant**. The purpose is two fold:

* Provide cross-platform installation of the DTaaS application. Any operating
system supporting use of vagrant software utility can support installation of
the DTaaS software.
* Provide cross-platform installation of the DTaaS application. Any operating system supporting use of vagrant software utility can support installation of the DTaaS software.
* Create a ready to use development environment for code contributors.

There are two scripts in this directory:
Expand All @@ -17,10 +15,10 @@ There are two scripts in this directory:
| `developer.sh` | developer installation | :x: |

If you are installing the DTaaS for developers, the default installation
caters to your needs. You can skip the next step and continue with the
caters to your needs. You can skip the next step and continue with the
creation of vagrant box.

If you are a developer and would like additional software installed, you need
If you are a developer and would like additional software installed, you need
to modify `Vagrantfile`. The existing `Vagrantfile` has two lines:

```sh
Expand All @@ -44,7 +42,7 @@ This vagrant box installed for users will have the following items:
* influxdb v2.7
* grafana v10.1
* rabbitmq v3-management
* eclipse-mosquitto (mqtt) v2
* eclipse-mosquitto (mqtt) v2

This vagrant box installed for developers will have
the following items additional items:
Expand Down
3 changes: 2 additions & 1 deletion deploy/vagrant/make_boxes/dtaas/provision.sh
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,8 @@ then
gpg --dearmor -o /etc/apt/keyrings/nodesource.gpg
fi
NODE_MAJOR=18
printf "deb [signed-by=/etc/apt/keyrings/nodesource.gpg] https://deb.nodesource.com/node_$NODE_MAJOR.x nodistro main" | \
printf "deb [signed-by=/etc/apt/keyrings/nodesource.gpg] \
https://deb.nodesource.com/node_%s.x nodistro main" "$NODE_MAJOR" | \
tee /etc/apt/sources.list.d/nodesource.list
apt-get update
apt-get install -y nodejs
Expand Down
3 changes: 2 additions & 1 deletion deploy/vagrant/make_boxes/dtaas/user.sh
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,8 @@ then
gpg --dearmor -o /etc/apt/keyrings/nodesource.gpg
fi
NODE_MAJOR=18
printf "deb [signed-by=/etc/apt/keyrings/nodesource.gpg] https://deb.nodesource.com/node_$NODE_MAJOR.x nodistro main" | \
printf "deb [signed-by=/etc/apt/keyrings/nodesource.gpg] \
https://deb.nodesource.com/node_%s.x nodistro main" "$NODE_MAJOR" | \
tee /etc/apt/sources.list.d/nodesource.list
apt-get update
apt-get install -y nodejs
Expand Down
10 changes: 4 additions & 6 deletions docs/admin/vagrant/base-box.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,7 @@ This README provides instructions on creating a custom Operating System
virtual disk for running the DTaaS software. The virtual disk is managed
by **vagrant**. The purpose is two fold:

* Provide cross-platform installation of the DTaaS application. Any operating
system supporting use of vagrant software utility can support installation of
the DTaaS software.
* Provide cross-platform installation of the DTaaS application. Any operating system supporting use of vagrant software utility can support installation of the DTaaS software.
* Create a ready to use development environment for code contributors.

There are two scripts in this directory:
Expand All @@ -17,10 +15,10 @@ There are two scripts in this directory:
| `developer.sh` | developer installation | :x: |

If you are installing the DTaaS for developers, the default installation
caters to your needs. You can skip the next step and continue with the
caters to your needs. You can skip the next step and continue with the
creation of vagrant box.

If you are a developer and would like additional software installed, you need
If you are a developer and would like additional software installed, you need
to modify `Vagrantfile`. The existing `Vagrantfile` has two lines:

```sh
Expand All @@ -44,7 +42,7 @@ This vagrant box installed for users will have the following items:
* influxdb v2.7
* grafana v10.1
* rabbitmq v3-management
* eclipse-mosquitto (mqtt) v2
* eclipse-mosquitto (mqtt) v2

This vagrant box installed for developers will have
the following items additional items:
Expand Down
4 changes: 3 additions & 1 deletion script/env.sh
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,10 @@ then
sudo gpg --dearmor -o /etc/apt/keyrings/nodesource.gpg
fi
NODE_MAJOR=18
printf "deb [signed-by=/etc/apt/keyrings/nodesource.gpg] https://deb.nodesource.com/node_$NODE_MAJOR.x nodistro main" | \
printf "deb [signed-by=/etc/apt/keyrings/nodesource.gpg] \
https://deb.nodesource.com/node_%s.x nodistro main" "$NODE_MAJOR" | \
sudo tee /etc/apt/sources.list.d/nodesource.list

sudo apt-get update
sudo apt-get install -y nodejs
sudo npm install -g [email protected]
Expand Down

0 comments on commit 6d464da

Please sign in to comment.