GitHub / databricks/scala-style-guide issues and pull requests
#91 - Separation of guide into 2 parts
Issue -
State: open - Opened by zubtsov about 1 year ago
#90 - Add detailed instructions on comments style
Pull Request -
State: open - Opened by xinglin-zhao over 1 year ago
#89 - Update README-ZH.md
Pull Request -
State: open - Opened by feiyx almost 2 years ago
#88 - Update README-ZH.md
Pull Request -
State: open - Opened by feiyx about 2 years ago
#87 - Do we have any spark scala syntax to read edi files into data frame
Issue -
State: open - Opened by sapthagirik about 2 years ago
#86 - Do not monadic-chain with an if-else block
Pull Request -
State: closed - Opened by HyukjinKwon almost 3 years ago
- 1 comment
#85 - Make it clear that there must be a space between class declaration and curly braces
Issue -
State: open - Opened by panbingkun about 3 years ago
#84 - Add Korean translation of #79
Pull Request -
State: closed - Opened by HyukjinKwon over 3 years ago
- 2 comments
#83 - Add section in scalastyle for new linter check
Pull Request -
State: closed - Opened by chadward-db almost 4 years ago
#82 - Missing "t" in the last section
Issue -
State: open - Opened by Timelessprod almost 4 years ago
#81 - Clarifying whitespace guidelines for classes
Issue -
State: open - Opened by MrPowers over 4 years ago
#80 - Add an example for function naming.
Pull Request -
State: closed - Opened by markj-db about 5 years ago
#79 - Update intro
Pull Request -
State: closed - Opened by rxin over 5 years ago
#78 - Chinese translation for recent updates
Pull Request -
State: closed - Opened by jiangr over 5 years ago
- 1 comment
#77 - Discourage to use symbol literals
Pull Request -
State: closed - Opened by HyukjinKwon over 5 years ago
- 2 comments
#76 - Add Korean translation for #75
Pull Request -
State: closed - Opened by HyukjinKwon over 5 years ago
- 2 comments
#75 - Clarification for enumerations
Pull Request -
State: closed - Opened by MaxGekk over 5 years ago
- 5 comments
#74 - Prefer curly brackets and dangling parentheses
Pull Request -
State: closed - Opened by AlexisDeschamps almost 6 years ago
- 1 comment
#73 - Split init
Pull Request -
State: closed - Opened by obar1 about 6 years ago
- 1 comment
#72 - init
Pull Request -
State: closed - Opened by obar1 about 6 years ago
- 1 comment
#71 - Merge pull request #1 from databricks/master
Pull Request -
State: closed - Opened by xliuqq over 6 years ago
#70 - Constants Naming Convention
Issue -
State: open - Opened by Subfire over 6 years ago
#69 - Add initial scalastyle config
Pull Request -
State: closed - Opened by aaabramov almost 7 years ago
#68 - Clarify blanklines between objects in Blank Lines section
Pull Request -
State: closed - Opened by HyukjinKwon about 7 years ago
- 4 comments
#67 - Fixes minor typo in example
Pull Request -
State: closed - Opened by gvdent about 7 years ago
- 1 comment
#66 - add Chinese translation for #64
Pull Request -
State: closed - Opened by Hawstein about 7 years ago
- 1 comment
#65 - Update README-KO.md
Pull Request -
State: closed - Opened by goungoun over 7 years ago
- 2 comments
#64 - Clarify each argument in each line, two lines limit and add invocation style guide
Pull Request -
State: closed - Opened by HyukjinKwon over 7 years ago
- 6 comments
#63 - fix README-KO.md
Pull Request -
State: closed - Opened by oeegee almost 8 years ago
- 6 comments
#62 - Search and replace: elasped -> elapsed
Pull Request -
State: closed - Opened by flyrev almost 8 years ago
#61 - Update README-KO.md
Pull Request -
State: closed - Opened by flyrev almost 8 years ago
#60 - Update README-ZH.md
Pull Request -
State: closed - Opened by flyrev almost 8 years ago
#59 - Update README.md (typo)
Pull Request -
State: closed - Opened by flyrev almost 8 years ago
- 3 comments
#58 - Multi-line vs single-line method and class constructor declarations and invocations
Issue -
State: open - Opened by jkbradley about 8 years ago
- 5 comments
#57 - Bump up the number of contributors
Pull Request -
State: closed - Opened by HyukjinKwon about 8 years ago
- 2 comments
#56 - Update README-ZH.md
Pull Request -
State: closed - Opened by ukari about 8 years ago
- 1 comment
#55 - Add Korean translation for #54
Pull Request -
State: closed - Opened by HyukjinKwon over 8 years ago
- 10 comments
#54 - Prefer existing well-tested methods over reinventing the wheel
Pull Request -
State: closed - Opened by zsxwing over 8 years ago
- 4 comments
#53 - Fix intl table of contents
Pull Request -
State: closed - Opened by MrPowers over 8 years ago
- 1 comment
#52 - Clarify that private variables should not be prefixed with an underscore
Pull Request -
State: open - Opened by MrPowers over 8 years ago
- 3 comments
#51 - Fix the table of contents numbering
Pull Request -
State: closed - Opened by MrPowers over 8 years ago
- 1 comment
#50 - add Chinese translation for #47
Pull Request -
State: closed - Opened by Hawstein over 8 years ago
- 1 comment
#49 - Automated checks for CI ?
Issue -
State: closed - Opened by chrissng over 8 years ago
- 3 comments
#48 - Add Korean translation for #47
Pull Request -
State: closed - Opened by HyukjinKwon over 8 years ago
- 4 comments
#47 - Add basic whitespace rules
Pull Request -
State: closed - Opened by MrPowers over 8 years ago
- 2 comments
#46 - Discourage changing external API's method signature for binary compatibility
Issue -
State: closed - Opened by HyukjinKwon over 8 years ago
- 3 comments
#45 - add Chinese translation for #43
Pull Request -
State: closed - Opened by Hawstein over 8 years ago
- 1 comment
#44 - Add Korean translation for #43
Pull Request -
State: closed - Opened by HyukjinKwon over 8 years ago
- 4 comments
#43 - Added testing section and section on intercepting exceptions
Pull Request -
State: closed - Opened by GregOwen over 8 years ago
- 8 comments
#42 - add Chinese translation for #39 and #40
Pull Request -
State: closed - Opened by Hawstein over 8 years ago
- 1 comment
#41 - Add Korean translation for #39 and #40
Pull Request -
State: closed - Opened by HyukjinKwon over 8 years ago
- 2 comments
#40 - Don't pattern match on all arguments unless needed
Pull Request -
State: closed - Opened by rxin almost 9 years ago
- 3 comments
#39 - Case classes and immutability
Pull Request -
State: closed - Opened by rxin almost 9 years ago
- 5 comments
#38 - avoid return if not neccesary, more idiomatic
Pull Request -
State: closed - Opened by 123avi almost 9 years ago
- 1 comment
#37 - no underscores in variable names
Pull Request -
State: closed - Opened by will-bennett-zocdoc almost 9 years ago
- 1 comment
#36 - Encourage to use `==` instead of `===` for writing unit tests.
Issue -
State: closed - Opened by HyukjinKwon about 9 years ago
- 9 comments
#35 - fix wrong choice of words (korean readme)
Pull Request -
State: closed - Opened by dexter-hwang about 9 years ago
- 2 comments
#34 - translate variable-naming-conversion section
Pull Request -
State: closed - Opened by Hawstein about 9 years ago
- 1 comment
#33 - Add variable naming section for Korean and fix miscellaneous inconsistent things for guidelines
Pull Request -
State: closed - Opened by HyukjinKwon about 9 years ago
- 5 comments
#32 - Variable Naming Convention
Pull Request -
State: closed - Opened by rxin about 9 years ago
- 4 comments
#31 - translate anonymous section and do some minor fix
Pull Request -
State: closed - Opened by Hawstein about 9 years ago
- 1 comment
#30 - Added Anonymous Methods section
Pull Request -
State: closed - Opened by HyukjinKwon about 9 years ago
- 8 comments
#29 - Revisit non-scalaish recommendations
Issue -
State: closed - Opened by ghost about 9 years ago
- 8 comments
#28 - Update README.md - Code sample correctness for incrementing values
Pull Request -
State: closed - Opened by enragedginger about 9 years ago
- 1 comment
#27 - fixed a typo
Pull Request -
State: closed - Opened by aseigneurin over 9 years ago
- 1 comment
#26 - Fix incorrect translation.
Pull Request -
State: closed - Opened by dongjoon-hyun over 9 years ago
- 3 comments
#25 - Thoughts about Option.get
Issue -
State: open - Opened by mgyucht over 9 years ago
- 2 comments
#24 - minor changes
Pull Request -
State: closed - Opened by keypointt over 9 years ago
- 2 comments
#23 - Adding Korean version of guide line.
Pull Request -
State: closed - Opened by HyukjinKwon over 9 years ago
- 7 comments
#22 - Adding Korean version of guide line.
Issue -
State: closed - Opened by HyukjinKwon over 9 years ago
- 16 comments
#21 - Update apply method section and document history in Chinese translation
Pull Request -
State: closed - Opened by Hawstein over 9 years ago
- 1 comment
#20 - Update apply method section in Chinese translation
Issue -
State: closed - Opened by rxin over 9 years ago
- 1 comment
#19 - apply on companion objects should return the companion class
Pull Request -
State: closed - Opened by rxin over 9 years ago
#18 - Translate into Chinese
Pull Request -
State: closed - Opened by Hawstein over 9 years ago
- 5 comments
#17 - Synchronized definitions which exceed 100 columns
Issue -
State: open - Opened by mgyucht over 9 years ago
- 2 comments
#16 - PascalCase instead of camelCase with first letter capitalized
Pull Request -
State: closed - Opened by alexandrnikitin almost 10 years ago
- 1 comment
#15 - avoid referring to methods as functions, since they are different concepts
Pull Request -
State: closed - Opened by stew almost 10 years ago
- 1 comment
#14 - Weaken some of the "Do NOT" to "Avoid"
Pull Request -
State: closed - Opened by rxin almost 10 years ago
- 1 comment
#13 - Trim trailing spaces.
Pull Request -
State: closed - Opened by rxin almost 10 years ago
#12 - Is there any editorconf settings for the IDEs?
Issue -
State: open - Opened by shadow-fox about 10 years ago
- 12 comments
#11 - Added section on override modifier and document history tracking
Pull Request -
State: closed - Opened by rxin about 10 years ago
#10 - Question on: "Avoid defining apply methods on classes"
Issue -
State: closed - Opened by tomer-ben-david about 10 years ago
- 2 comments
#9 - Minor doc fix.
Pull Request -
State: closed - Opened by concretevitamin over 10 years ago
- 1 comment
#8 - Change wording around concurrent.Map
Pull Request -
State: closed - Opened by dragos over 10 years ago
- 2 comments
#7 - Fixed link to URL.equals in readme
Pull Request -
State: closed - Opened by sksamuel over 10 years ago
- 1 comment
#6 - ConcurrentMap qualifications
Issue -
State: closed - Opened by dragos over 10 years ago
- 1 comment
#5 - Update README.md
Pull Request -
State: closed - Opened by stephentu over 10 years ago
- 1 comment
#4 - Fix grammar
Pull Request -
State: closed - Opened by mrefish over 10 years ago
- 1 comment
#3 - fix typo/spelling
Pull Request -
State: closed - Opened by SethTisue over 10 years ago
- 1 comment
#2 - Change currentTimeInMillis to currentTimeMillis
Pull Request -
State: closed - Opened by kayousterhout over 10 years ago
- 2 comments
#1 - Some edits, mainly grammatical
Pull Request -
State: closed - Opened by aarondav over 10 years ago