Before you do a pull request, you should always file an issue and make sure the package maintainer agrees that it’s a problem, and is happy with your basic proposal for fixing it. We don’t want you to spend a bunch of time on something that we don’t think is a good idea.
67 |Additional requirements for pull requests:
68 |Adhere to the Developer Guidelines as well as the OHDSI Code Style.
If possible, add unit tests for new functionality you add.
Restrict your pull request to solving the issue at hand. Do not try to ‘improve’ parts of the code that are not related to the issue. If you feel other parts of the code need better organization, create a separate issue for that.
Make sure you pass R check without errors and warnings before submitting.
Always target the develop branch, and make sure you are up-to-date with the develop branch.
SqlRender
130 |R/HelperFunctions.R
64 | camelCaseToSnakeCase.RdConvert a camel case string to snake case
69 |camelCaseToSnakeCase(string)A string
86 |camelCaseToSnakeCase("exposureConceptId1")
91 | #> [1] "exposure_concept_id_1"
92 |
93 | R/HelperFunctions.R
64 | camelCaseToTitleCase.RdConvert a camel case string to title case
69 |camelCaseToTitleCase(string)A string
86 |camelCaseToTitleCase("exposureConceptId1")
91 | #> [1] "Exposure Concept Id 1"
92 |
93 | R/RenderSql.R
65 | getTempTablePrefix.RdGet the prefix used for emulated temp tables for DBMSs that do not support temp tables (e.g. Oracle, 70 | BigQuery).
71 |getTempTablePrefix()The prefix string.
80 |getTempTablePrefix()
85 | #> [1] "l3470uc6"
86 | R/ShinyApps.R
64 | launchSqlRenderDeveloper.RdLaunch the SqlRender Developer Shiny app
69 |launchSqlRenderDeveloper(launch.browser = TRUE)Launches a Shiny app that allows the user to develop SQL and see how it translates to the supported 87 | dialects.
88 |R/HelperFunctions.R
64 | snakeCaseToCamelCase.RdConvert a snake case string to camel case
69 |snakeCaseToCamelCase(string)A string
86 |snakeCaseToCamelCase("exposure_concept_id_1")
91 | #> [1] "exposureConceptId1"
92 |
93 | This function is for Spark connections only, 70 | it handles insert commands, as Spark cannot handle inserts with aliased or subset columns.
71 |sparkHandleInsert(sql, connection)A sql string with INSERT command modified to contain the full column list, padded with NULLS as needed.
92 |R/HelperFunctions.R
64 | supportsJava8.RdTests Java virtual machine (JVM) java.version system property to check if version >= 8.
69 |supportsJava8()Returns TRUE if JVM supports Java >= 8.
78 |supportsJava8()
83 | #> [1] TRUE
84 |