-
Notifications
You must be signed in to change notification settings - Fork 492
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Versions Bump #184
base: master
Are you sure you want to change the base?
Versions Bump #184
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Your PR does not pass CI build.
Also sign off is missing
@@ -15,57 +15,57 @@ EXPOSE 3000 | |||
# installs development files for node-gyp so that npm install won't have to | |||
# wait for them on the first native module installation. | |||
RUN export DEBIAN_FRONTEND=noninteractive && \ | |||
useradd --system \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why formatting here is changed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this was the auto-formatting
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Your PR does not pass CI build.
Can you provide more information, or main error it's missing signing?
also sign off is missing
Okay, 5 mins
@@ -3,7 +3,7 @@ version: "3" | |||
services: | |||
rethinkdb: | |||
container_name: rethinkdb | |||
image: rethinkdb:2.3 | |||
image: rethinkdb:latest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should use fixed version.
It is ok to update to 2.4.1 but not to latest
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay
@koral-- can you reply, why in docker file you don't use nvm and latest nodejs version? |
STF won't build with latest node version. |
okay, I will try to fix it |
Updated to latest version of RDB. With old version project is not started Removed yarn.lock Version dump Version dump
This reverts commit e17007f.
This reverts commit e17007f.
# Conflicts: # package.json
@koral, can you describe the actual codebase status? I would suggest separate frontend from backend and rewrite it to nestsjs and modern angular |
I'm not a JS expert but will try to answer.
Correct.
Which method?
LGTM at a glance. Upgrading Angular is perfectly reasonable. |
I don't know how to change authentication method, but I always get auth_mock in URL
NestJS has express/fastify framework in the core, it's just a wrapper that will provide typization. And this framework has is angular-like code style. And I saw repos stf_ios_support and stf-ios-provider, can you describe what is it? I don't fully understand the structure of the project |
Regarding auth there are a few methods available: look at https://github.com/DeviceFarmer/stf/blob/master/doc/DEPLOYMENT.md#stf-authservice Regarding NestJS fine for me, if you will be able to rewrite using it, PRs are welcome. Regarding iOS-related repos this is an unfinished @nanoscopic's work about iOS support. |
Updated versions of database to latest. And Ubuntu from 16 to 21