Skip to content

puncsky/coding-challenge-tools

 
 

Folders and files

NameName
Last commit message
Last commit date

Latest commit

 

History

39 Commits
 
 
 
 

Repository files navigation

Coding challenge or existing code?

The coding challenge is optional if you already have some code that you're proud of and can share with us.

Existing code

If you have existing code, please follow the following guidelines:

  • Include a link to the hosted repository (e.g. Github, Bitbucket...). We cannot review archives or single files.
  • The repo should include a README that follows the principles described below In particular, please make sure to include high-level explanation about what the code is doing.
  • Ideally, the code you're providing:
    • Has been written by you alone. If not, please tell us which part you wrote and are most proud of in the README.
    • Is leveraging web technologies.
    • Is deployed and hosted somewhere.

Readme

Regardless of whether it's your own code or our coding challenge, write your README as if it was for a production service. Include the following items:

  • Description of the problem and solution.
  • Whether the solution focuses on back-end, front-end or if it's full stack.
  • Reasoning behind your technical choices, including architectural.
  • Trade-offs you might have made, anything you left out, or what you might do differently if you were to spend additional time on the project.
  • Link to other code you're particularly proud of.
  • Link to your resume or public profile.
  • Link to to the hosted application where applicable.

How we review

Your application will be reviewed by at least three of our engineers. We do take into consideration your experience level.

We value quality over feature-completeness. It is fine to leave things aside provided you call them out in your project's README. The goal of this code sample is to help us identify what you consider production-ready code. You should consider this code ready for final review with your colleague, i.e. this would be the last step before deploying to production.

The aspects of your code we will assess include:

  • Architecture: how clean is the separation between the front-end and the back-end?
  • Clarity: does the README clearly and concisely explains the problem and solution? Are technical tradeoffs explained?
  • Correctness: does the application do what was asked? If there is anything missing, does the README explain why it is missing?
  • Code quality: is the code simple, easy to understand, and maintainable? Are there any code smells or other red flags? Does object-oriented code follows principles such as the single responsibility principle? Is the coding style consistent with the language's guidelines? Is it consistent throughout the codebase?
  • Security: are there any obvious vulnerability?
  • Testing: how thorough are the automated tests? Will they be difficult to change if the requirements of the application were to change? Are there some unit and some integration tests?
    • We're not looking for full coverage (given time constraint) but just trying to get a feel for your testing skills.
  • UX: is the web interface understandable and pleasing to use? Is the API intuitive?
  • Technical choices: do choices of libraries, databases, architecture etc. seem appropriate for the chosen application?

Bonus point (those items are optional):

  • Scalability: will technical choices scale well? If not, is there a discussion of those choices in the README?
  • Production-readiness: does the code include monitoring? logging? proper error handling?

Coding Challenge

Guidelines can be found here.

About

Uber's tools team coding challenge

Resources

Stars

Watchers

Forks

Releases

No releases published

Packages

No packages published